Skip to content
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 16 forward compatibility #48

Merged
merged 1 commit into from
Nov 23, 2022
Merged

Conversation

aaronadamsCA
Copy link
Contributor

@aaronadamsCA aaronadamsCA commented Jan 17, 2022

Resolves #47.

I applied this locally as a monkey patch, and it works perfectly. I haven't identified any other issues with GraphQL forward compat.

(This patch was so simple that I literally typed it into GitHub, so just make sure I didn't make any silly typos.)

Make compatible with GraphQL 16
@aaronadamsCA aaronadamsCA changed the title GraphQL 16 compatibility GraphQL 16 forward compatibility Jan 17, 2022
@aaronadamsCA aaronadamsCA marked this pull request as ready for review January 17, 2022 10:39
@AbdallahAbis
Copy link

AbdallahAbis commented Mar 25, 2022

@LekoArts @aaronadamsCA is there a reason why this is not merged yet? trying to use the toolkit but I'm getting the same error on #47, I have tested the generated fragments against the remote Graphql and they work, but since compileNodeQueries fails I couldn't test the compiled queries.

Screen Shot 2022-03-25 at 03 15 55

@aaronadamsCA
Copy link
Contributor Author

@AbdallahAbis I patched the package locally, which you can also do with yarn patch if available, and patch-package otherwise. I can't speak on merge timelines, unfortunately, I'm just a community contributor.

@LekoArts LekoArts merged commit 50f2c62 into gatsbyjs:master Nov 23, 2022
@LekoArts
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GraphQL 16 compatibility
3 participants