-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add work items section #4
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.
woo! thanks for putting this together :)
PRs will be linked here. | ||
- Spec Changes | ||
- Official Libraries | ||
- GraphQL.js: https://github.com/graphql/graphql-js/pull/2824 |
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.
let's double check with @xuewei8910 to make sure this PR is ready for external eyes (I think he mentioned it needed to be updated with latest)
rfcs/QueryLevelNullability.md
Outdated
- [Apollo Android](https://github.com/apollographql/apollo-android) | ||
- Code Gen | ||
- Cache | ||
- [Apollo iOS](https://github.com/apollographql/apollo-ios) | ||
- Code Gen | ||
- Cache | ||
- [GraphQL Code Generator by The Guild](https://github.com/dotansimha/graphql-code-generator) |
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.
just wondering: do we need to implement a change for both of these libraries & all 3 platforms for the proof of concept?
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.
If I'm reading the contribution doc correctly, it looks like we need an implementation in GraphQL.js, and otherwise need to do any implementations needed to address concerns during the review process. Let's confirm during he meeting?
https://github.com/graphql/graphql-spec/blob/main/CONTRIBUTING.md
Co-authored-by: Liz Jakubowski <[email protected]>
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!
No description provided.