Skip to content

type/introspection: add missing __Directive.args(includeDeprecated:) #3273

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

Merged
merged 1 commit into from
Sep 24, 2021

Conversation

IvanGoncharov
Copy link
Member

Fixes #2895
Fixes #3046

@falkenhawk
Copy link

I believe this also fixes #3048?

@IvanGoncharov
Copy link
Member Author

@falkenhawk Thanks a lot 👍

spawnia pushed a commit to webonyx/graphql-php that referenced this pull request Jul 28, 2025
Adds the missing includeDeprecated argument to `__Directive.args` field
to match GraphQL.js implementation (graphql/graphql-js#3273).

The argument allows filtering deprecated directive arguments from
introspection results when set to false (default behavior).

This notably fixes schema generation with `graphql-codegen` when using
the "inputValueDeprecation" option (see dotansimha/graphql-code-generator#9659).

Includes tests for deprecated argument filtering that I didn't find
in GraphQL.js.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: bug fix 🐞 requires increase of "patch" version number
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unknown argument "includeDeprecated" on field "__Directive.args"
2 participants