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

Apply escaping to input object parameter names #2556

Closed
wants to merge 1 commit into from
Closed

Apply escaping to input object parameter names #2556

wants to merge 1 commit into from

Conversation

puls
Copy link
Contributor

@puls puls commented Oct 8, 2022

I'm not sure if this is sufficient, but I have some input objects with fields named in and this change allows my generated code to compile.

I'm not sure if this is sufficient, but I have some input objects with fields named `in` and this change allows my generated code to compile.
@apollo-cla
Copy link

@puls: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

@netlify
Copy link

netlify bot commented Oct 8, 2022

👷 Deploy request for apollo-ios-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 3e473ae

@AnthonyMDev
Copy link
Contributor

Thanks for this! I'm going to write some unit tests for this before merging in. Be on the lookout for version 1.0.1 very soon with the fix!

@AnthonyMDev
Copy link
Contributor

Closing in favor of #2561! Thanks so much for this fix!

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.

3 participants