-
Notifications
You must be signed in to change notification settings - Fork 10.3k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix: ensure graphql key (starting with numeric) is valid #7913
Conversation
Hey @eLod, could you also add some tests here - to this inline fixture gatsby/packages/gatsby/src/schema/__tests__/infer-graphql-type-test.js Lines 58 to 111 in 7132443
|
@pieh sure, just to be sure: so you want me to add a value to that
do you want me to add multiple keys or just a single one? |
Yeah, this sounds good! 👍 |
Looking good! Just waiting for appveyor to finish before merging :) |
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 @eLod!
Holy buckets, @eLod — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
handling keys that start with numeric, see #4000