-
Notifications
You must be signed in to change notification settings - Fork 155
docs: graphql validations guide #284
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
docs: graphql validations guide #284
Conversation
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.
thanks for putting this together! added some comments
title: GraphQL Validations migration guide | ||
--- | ||
|
||
Soon `graph-node` will support 100% coverage of the [GraphQL Validations specification](https://spec.graphql.org/June2018/#sec-Validation). |
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.
We should align on timing and be clear in this document what the expectations are for:
- Hosted Service
- A release on the network
e6f0a9e
to
fffd4d1
Compare
fffd4d1
to
c75a526
Compare
c75a526
to
2858f04
Compare
📦 Next.js Bundle AnalysisThis analysis was generated by the next.js bundle analysis action 🤖
|
Page | Size (compressed) |
---|---|
global |
1.42 MB (🟡 +45 B) |
Details
The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.
Any third party scripts you have added directly to your app using the <script>
tag are not accounted for in this analysis
If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!
New Page Added
The following page was added to the bundle from the code in this PR:
Page | Size (compressed) | First Load | % of Budget (350 KB ) |
---|---|---|---|
/en/release-notes/graphql-validations-migration-guide |
9.11 KB |
1.42 MB | 416.80% |
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.
Thank you for working on this 🙌
Left a few comments 🙏
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
website/pages/en/release-notes/graphql-validations-migration-guide.mdx
Outdated
Show resolved
Hide resolved
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.
LGTM Thank You!
closes #282