Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Show loading, polling, and error status for queries and mutations #1429

Open
wants to merge 89 commits into
base: main
Choose a base branch
from
Open

Show loading, polling, and error status for queries and mutations #1429

wants to merge 89 commits into from

Conversation

jerelmiller
Copy link
Member

Adds the ability to see whether a query or mutation is in flight and show their associated errors.

Screenshot 2024-06-26 at 1 12 29 AM Screenshot 2024-06-25 at 10 33 07 PM Screenshot 2024-06-25 at 10 33 12 PM Screenshot 2024-06-25 at 10 33 40 PM Screenshot 2024-06-25 at 10 37 37 PM Screenshot 2024-06-25 at 10 37 48 PM Screenshot 2024-06-25 at 11 22 48 PM Screenshot 2024-06-25 at 11 22 57 PM

@jerelmiller
Add networkStatus and error when getting queries
53ec040
@jerelmiller
Add additional types needed to get networkStatus and errors
b541b1b
@jerelmiller
Simplify some types and write data to cache
b559d75
@jerelmiller
Fix typo
9932c9e
@jerelmiller
Write additional data to cache
62c368d
@jerelmiller
Fix spinner className for sm size
0e3b859
@jerelmiller
Ensure ListItem content is full width
2f8db62
@jerelmiller
Show spinner when query is in flight
f98d592
@jerelmiller
Show error icon if query errored
10404e5
@jerelmiller
Allow QueryLayout.Title to accept className
168aaa2
@jerelmiller
Remove duplicat className
3af7276
@jerelmiller
Force font family in StatusBadge
153fef8
@jerelmiller
Add StatusBadge next to the title when fetching
4736f53
@jerelmiller
Allow an alert to be rendered as a button
db15ddb
@jerelmiller
Create an AlertDisclosure component
5114b8e
@jerelmiller
Forward ref to Alert
a8333d8
@jerelmiller
Rerun codegen
c954c34
@jerelmiller
Show error for query
2251757
@jerelmiller
Show path for graphql errors
40a1d2e
@jerelmiller
Tweak to format of error message
ed92e71
@jerelmiller
Add extensions to graphql errors
0371146
@jerelmiller
Send extensions with errors
1d03764
@jerelmiller
Change type of path to JSON
ebe2eb1
@jerelmiller
Show extensions in error
8d1f0a1
@jerelmiller
Simplify path display
0e7c529
@jerelmiller
Add QueryLayout.Content wrapper to allow more content inside it
686c2ea
@jerelmiller
Use updated layout structure for mutations
b39e982
@jerelmiller
Move query errors above query
c44b4d3
@jerelmiller
Swap the direction of the chevron in alerrt AlertDisclosure
572af4d
@jerelmiller
Change type of path for graphql error type
1f441d5
@jerelmiller
Add additional data to WatchedMutation type
4aa5081
@jerelmiller
Add additional mutation details to store
808aade
@jerelmiller
Fix height of mutation list item
4a29107
@jerelmiller
Show spinner when mutation is loading
97e95b1
@jerelmiller
Move error alert disclosure item to own component
b8547a4
@jerelmiller
Show status on mutation
19efbcc
@jerelmiller
Change type of error for mutations to allow for apollo errors
ec95f9a
@jerelmiller
Return serialized apollo error on mutation
0909490
@jerelmiller
Use serializeError helper
c82d2e7
@jerelmiller
Use const name for apollo error
f8e694b
@jerelmiller
Export serialized error types
4ad5171
@jerelmiller
Write mutation error
9a85bf2
@jerelmiller
Create ApolloErrorAlertDisclosurePanel
ce0bdb0
@jerelmiller
Create a SerializedErrorAlertDisclosureItem component
c9b0b1c
@jerelmiller
Use new disclosure item in alert panel
4fcc93c
@jerelmiller
Fix wrong message for errors
abf4422
@jerelmiller
Use new alert panel component for queries
50a113d
@jerelmiller
Add possibleTypes
e239bd1
@jerelmiller
Show errors for mutations
81a9b7f
@jerelmiller jerelmiller requested a review from a team as a code owner June 26, 2024 07:15
@relativeci RelativeCI
Copy link

relativeci bot commented Jun 26, 2024 •
edited
Loading

#621 Bundle Size — 1.29MiB (+0.8%).

fb06830(current) vs 62d8221 main#614(baseline)

Warning

Bundle contains 13 duplicate packages – View duplicate packages

Bundle metrics  Change 4 changes Regression 1 regression
                 Current
#621		      Baseline
#614
Regression  Initial JS 1.25MiB(+0.82%) 1.24MiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 92.41% 0%
No change  Chunks 5 5
No change  Assets 12 12
Change  Modules 1040(+1.17%) 1028
No change  Duplicate Modules 49 49
Change  Duplicate Code 3.92%(-0.51%) 3.94%
No change  Packages 162 162
No change  Duplicate Packages 10 10
Bundle size by type  Change 1 change Regression 1 regression
                 Current
#621		      Baseline
#614
Regression  JS 1.25MiB (+0.82%) 1.24MiB
No change  IMG 35.85KiB 35.85KiB
No change  HTML 810B 810B
No change  Other 778B 778B

Bundle analysis report Branch jerel/query-status Project dashboard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phryneas phryneas Awaiting requested review from phryneas

@alessbell alessbell Awaiting requested review from alessbell

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@jerelmiller