-
Notifications
You must be signed in to change notification settings - Fork 3
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
[GraphQL]: Add Relay support #26
Comments
hey @buffalojoec!
Is there any change needed inside the resolvers or loaders? From the specification in the provided link i can see that ID is a required field, shall that be the case for this PR as well; or shall it be optional? Thanks |
Hi @tibi77 thanks for reaching out! I think you've generalized it pretty well here, although we don't know how much of an effort it will be to be completely compliant with the Relay spec until we've started to uncover a few stones in the schema and their docs. For example, we might have to investigate pagination and cursor connections, but I think the ID and the tests are a good place to start. It's worth noting that we actually have suitable identifiers for each schema object.
I think we just need to hack around and see how practical it might be to either:
Both are pretty gross for their own reasons, but it won't become clear which is the better move until we've messed around with it up close. |
@buffalojoec opened a draft just for the transactions; curious if i m doing it correctly |
@tibi77 Try making sure you don't have a test validator running, and then from the workspace root run |
ohhh nice; thanks for that |
Motivation
The GraphQL package can be much easier to integrate into web applications if it adheres to the Relay specification and can be used by apps powered by Relay.
One of the biggest schema changes required to be Relay-compliant will be the use of an
ID
field.The text was updated successfully, but these errors were encountered: