-
Notifications
You must be signed in to change notification settings - Fork 324
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
Hydrogen-ui storefront API schema and types #1122
Merged
Merged
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
d126c74
Basic vitest setup
frehner fba4612
Get vitest working with react-testing-library
frehner 27ca05c
Finish tests, get all of them passing, and cleanup issues
frehner e6fa096
update docs
frehner 507f0a8
Merge branch 'v1.x-2022-07' into hydrogen-ui-tests
frehner 64fde6b
Update yarn.lock
frehner 610c369
restarting the eslint server fixed this issue
frehner 676a5d8
Exclude h-ui from Jest, and run h-ui's own tests
frehner 1025ee5
Remove react import now that I have vitest setup with Vite config
frehner 720168e
Update packages/hydrogen-ui/src/ExternalVideo.tsx
frehner daa81ca
Update packages/hydrogen-ui/src/ExternalVideo.tsx
frehner 86909ed
Update packages/hydrogen-ui/CONTRIBUTING.md
frehner f5471a3
saving progress
frehner a92855f
Update packages/hydrogen-ui/README.md
frehner 33153d3
feedback
frehner 3178435
Merge branch 'hydrogen-ui-tests' into hui-sfapi-types
frehner 1a91cf1
Add basic readme docs, rename file, get copy working
frehner 94c43a2
We can use the type keyword not that we're not using enums
frehner 80f0794
clarify documentation
frehner a7922d3
Merge branch 'v1.x-2022-07' into hui-sfapi-types
frehner 9978d16
Ignore the schema file for prettier
frehner 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
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
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 |
---|---|---|
@@ -0,0 +1,37 @@ | ||
overwrite: true | ||
schema: | ||
- https://hydrogen-preview.myshopify.com/api/2022-07/graphql.json: | ||
headers: | ||
X-Shopify-Storefront-Access-Token: 3b580e70970c4528da70c98e097c2fa0 | ||
content-type: application/json | ||
generates: | ||
# the base types that are generated | ||
src/storefront-api-types.d.ts: | ||
plugins: | ||
- add: | ||
content: | ||
- '/** ' | ||
- ' * THIS FILE IS AUTO-GENERATED, DO NOT EDIT' | ||
- ' * Based on Storefront API 2022-07' | ||
- ' * If changes need to happen to the types defined in this file, then generally the Storefront API needs to update, and then you can run `yarn graphql-types`' | ||
- ' * Except custom Scalars, which are defined in the `codegen.yml` file' | ||
- ' */' | ||
- '/* eslint-disable */' | ||
- typescript: | ||
useTypeImports: true | ||
# If a default type for a scalar isn't set, then instead of 'any' let's set to 'unknown' for better type safety. | ||
defaultScalarType: unknown | ||
useImplementingTypes: true | ||
enumsAsTypes: true | ||
# Define how the SFAPI's custom scalars map to TS types | ||
scalars: | ||
DateTime: string | ||
Decimal: string | ||
HTML: string | ||
Money: string | ||
URL: string | ||
# the schema file, which is the local representation of the graphql endpoint | ||
./storefront.schema.json: | ||
plugins: | ||
- 'introspection': | ||
minify: true |
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
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
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.
@cartogram What we talked about this morning