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

Fix #2895 inputValueDeprecation: true getIntrospectionQuery error #3046

Closed
wants to merge 3 commits into from

Conversation

davidgovea
Copy link

Issue #2895

  • Adds test to reproduce the issue
  • Made a fix attempt: Not sure if removing includeDeprecated from directive is the right approach, but tests are passing

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Apr 21, 2021

CLA Signed

The committers are authorized under a signed CLA.

@davidgovea davidgovea changed the title Fix inputValueDeprecation: true getIntrospectionQuery error Fix #2895 inputValueDeprecation: true getIntrospectionQuery error Apr 21, 2021
@znarf
Copy link

znarf commented Jul 30, 2021

Could this be considered by maintainers? We're waiting for this fix at Open Collective. Happy to help in any way!

@znarf
Copy link

znarf commented Aug 30, 2021

@IvanGoncharov no hurry but could you have a look and tell us what to expect about this PR?

@IvanGoncharov
Copy link
Member

@davidgovea Thanks for PR and sorry for the late review :(
It was actually a case of missing includeDeprecated argument on directive so I added it in #3273

@davidgovea
Copy link
Author

Thanks @IvanGoncharov!

@davidgovea davidgovea closed this Sep 24, 2021
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.

None yet

3 participants