-
Notifications
You must be signed in to change notification settings - Fork 90
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.
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
Catalog: GraphQL for packages #2552
Merged
Merged
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
a0c40d9
utils/useQuery: helper wrapper for urql useQuery
nl0 df316b0
dont try to load catalog settings in local mode
nl0 d5b1205
WIP graphql schema changes for packages and related stuff
nl0 12c667a
shared: update gql schema
nl0 daea162
graphqlrc: add JsonRecord scalar
nl0 320f8fb
shared: graphql schema: JsonDict -> JsonRecord
nl0 a9e9ab8
update generated gql stuff
nl0 a9e62f3
Bucket/PackageList: get data from graphql
nl0 1b06c3a
shared: gql schema: add package list order
nl0 9432e1b
gql generated files: regenerate to include order
nl0 ea27baa
PackageList: tsify, use gql
nl0 a985596
Bucket/requests: rm requests migrated to gql
nl0 4675ac3
cat: adjust gql schema
nl0 9a2b7be
PackageUpdateDialog: hash is optional
nl0 9f04060
adjust PackageList
nl0 ff5a080
PackageRevisions: gql, ts
nl0 3027bc3
rm obsolete requests
nl0 c2b9fec
PackageRevisions: useQuery
nl0 a17a4f7
Bucket/PackageList: useQuery
nl0 24c96f9
update gql schema
nl0 aefc3b1
update generated gql stuff
nl0 cccbfa0
Bucket/PackageRevisions: adjust for gql changes
nl0 b73d51c
Bucket/PackageTree: ts, gql, refactor
nl0 cee1a39
Bucket/requests: rm unused code
nl0 3b9d708
Bucket/errrors: rm unused code
nl0 c7e4840
Bucket/PackageTree: use gql for logical key resolution
nl0 396baeb
Bucket/requests: rm unused code
nl0 b743b74
PackageTree: fix some gql stuff
nl0 bee1268
PackageTree: fix gql stuff
nl0 01d99f9
fix requests
nl0 1ac0042
utils/GraphQL: keys for new types
nl0 438010b
update gql schema
nl0 fdf66fc
catalog: further work on packages gql stuff
nl0 02da532
show package folder size
nl0 a8fff9d
support relative urls for s3proxy and api gateway
nl0 65043c3
AWS/Credentials: handle missing Expiration
nl0 f26bde2
utils/useQuery: more descriptive type var names
nl0 c820588
address CR feedback
nl0 b83315c
rm PackageTree.js
nl0 72a98b6
gql schema: nullable totalEntries and totalBytes
nl0 5776624
regen gql model
nl0 8ebf590
PackageRevisions: support nullable totalEntries
nl0 d287367
support revisions with non-unique hashes
nl0 374e745
gql schema: Floats for sizes
nl0 4c170e9
regen gql stuff
nl0 4c01178
changelog entry
nl0 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,3 +19,4 @@ extensions: | |
scalars: | ||
Datetime: Date | ||
Json: utils/types#Json | ||
JsonRecord: utils/types#JsonRecord |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see multiple
undefined
in this module. Can we usenull
here? I think that it's more straightforward to usenull
, and leaveundefined
only when it's really undefinedThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here -- yes, but not everywhere (e.g. not as a
title
attribute for aspan
, since it only acceptsstring | undefined
)tbh i don't feel much difference in most cases 🤷 (with a notable exception of
JSON.strignify
)