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

Add function to check if a request is a GraphQL call. #1211

Merged
merged 1 commit into from
Aug 7, 2022

Conversation

EarthlingDavey
Copy link
Contributor

Specifically for wp-graphql plugin: https://github.com/wp-graphql/wp-graphql - 3.3k Github stars and 20k active installs.

Add function to check if a request is a GraphQL call, and, if GraphQL call then don't do redirect or set cookies.

Without this change, WPGraphQL does not function correctly. It's default endpoint of /graphql is redirected to /xx/graphql

…r wp-graphql plugin: https://github.com/wp-graphql/wp-graphql

And, if GraphQL call then don't do redirect or set cookies.
@herrvigg herrvigg added the core Core functionalities, including the admin section label Aug 3, 2022
Copy link
Collaborator

@herrvigg herrvigg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@herrvigg herrvigg merged commit f9556d5 into qtranslate:master Aug 7, 2022
@EarthlingDavey
Copy link
Contributor Author

Thanks bud. I appreciate your's and the other contributor's work 👊👊✌️

@herrvigg herrvigg added plugin: others Concerns integration with other plugins and removed core Core functionalities, including the admin section labels Aug 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin: others Concerns integration with other plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants