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(gatsby-source-wordpress): WPGraphQL 1.13.0 compatibility #37134

Merged
merged 34 commits into from
Dec 2, 2022

Conversation

TylerBarnes
Copy link
Contributor

@TylerBarnes TylerBarnes commented Nov 30, 2022

Fixes #37114

WPGraphQL v1.13.0 included a breaking change in a minor version because for most consumers they wouldn't notice. For gatsby-source-wordpress it was very much a huge breaking change as it broke many of the assumptions this plugin made about the WPGraphQL schema.

I had to change a lot of code to make the plugin keep working with older versions of WPGraphQL and the new version, I'll add some comments to the files view instead of writing it all here

In this PR I kept the integration tests WPGraphQL version to an older version to make sure these changes are backwards compatible. In #37146 I upgrade the itests version to the 1.13.4

This change is also released as a canary gatsby-source-wordpress@7.3.0-alpha-wpgql-thirteen.20

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Nov 30, 2022
@TylerBarnes TylerBarnes removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Nov 30, 2022
@LekoArts LekoArts added topic: source-wordpress Related to Gatsby's integration with WordPress topic: source-plugins Relates to the Gatsby source plugins (e.g. -filesystem) labels Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: source-plugins Relates to the Gatsby source plugins (e.g. -filesystem) topic: source-wordpress Related to Gatsby's integration with WordPress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[gatsby-source-wordpress] TypeError: Cannot destructure property 'fields' of 'nodesType' as it is undefined
6 participants