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

Remove support for positional args in graphql/execute/subscribe func #2904

Merged
merged 1 commit into from Feb 1, 2021

Conversation

IvanGoncharov
Copy link
Member

Closes #2301

@IvanGoncharov IvanGoncharov added the PR: breaking change 💥 implementation requires increase of "major" version number label Jan 31, 2021
@IvanGoncharov IvanGoncharov merged commit c0ff68e into graphql:main Feb 1, 2021
@IvanGoncharov IvanGoncharov deleted the pr_branch branch February 1, 2021 13:39
IvanGoncharov added a commit to IvanGoncharov/graphql-js that referenced this pull request Feb 20, 2021
IvanGoncharov added a commit to IvanGoncharov/graphql-js that referenced this pull request Mar 24, 2021
Removal of positional args was done in graphql#2904 but TS typings were not updated.
IvanGoncharov added a commit that referenced this pull request Mar 24, 2021
Removal of positional args was done in #2904 but TS typings were not updated.
benjamn added a commit to apollographql/react-apollo-error-template that referenced this pull request Nov 22, 2021
Similar to changes made by @brainkim in
apollographql/apollo-client#8997, this commit
fixes the React Apollo template application when built with graphql@16,
since support for passing positional arguments was removed in
graphql/graphql-js#2904.
benjamn added a commit to apollographql/react-apollo-error-template that referenced this pull request Nov 22, 2021
Similar to changes made by @brainkim in
apollographql/apollo-client#8997, this commit
fixes the React Apollo template application when built with graphql@16,
since support for passing positional arguments was removed in
graphql/graphql-js#2904.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: breaking change 💥 implementation requires increase of "major" version number
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant