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):issue #29535 not finished createSchemaCu… #29554

Merged
merged 2 commits into from
Feb 23, 2021
Merged

fix(gatsby-source-wordpress):issue #29535 not finished createSchemaCu… #29554

merged 2 commits into from
Feb 23, 2021

Conversation

lwz7512
Copy link
Contributor

@lwz7512 lwz7512 commented Feb 17, 2021

fix issue #29535

Description

There is a bug in gatsby-source-wordpress plugin version 4.0.1 that cause the Wordpress blog build failure using developer's Graphql url. It turned out because of null value of transformedFields in build-types.js.

Documentation

no

Related Issues

#29535

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Feb 17, 2021
@LekoArts LekoArts added topic: source-wordpress Related to Gatsby's integration with WordPress and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Feb 22, 2021
@LekoArts
Copy link
Contributor

Hi, thanks for the PR! Please revert the changed yarn.lock file as that shouldn't be part of this PR. Thanks!

@TylerBarnes
Copy link
Contributor

Just tried this out and it works great 👍 thanks @lwz7512 ! Once you make the change requested above this is ready.

@TylerBarnes TylerBarnes self-assigned this Feb 22, 2021
@lwz7512
Copy link
Contributor Author

lwz7512 commented Feb 23, 2021

Hi, thanks for the PR! Please revert the changed yarn.lock file as that shouldn't be part of this PR. Thanks!

Reverted! Thank you for reminding!

@ascorbic ascorbic added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Feb 23, 2021
@gatsbybot gatsbybot merged commit 308d418 into gatsbyjs:master Feb 23, 2021
ascorbic pushed a commit that referenced this pull request Feb 23, 2021
#29554)

Co-authored-by: gatsbybot <mathews.kyle+gatsbybot@gmail.com>
(cherry picked from commit 308d418)
@lwz7512 lwz7512 deleted the fix/gatsby-source-wordpress branch February 23, 2021 20:56
ascorbic added a commit that referenced this pull request Feb 24, 2021
#29554) (#29712)

Co-authored-by: gatsbybot <mathews.kyle+gatsbybot@gmail.com>
(cherry picked from commit 308d418)

Co-authored-by: liwenzhi <lwz7512@gmail.com>
Co-authored-by: Matt Kane <matt@gatsbyjs.com>
@ascorbic
Copy link
Contributor

Published in gatsby-source-wordpress@4.0.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes topic: source-wordpress Related to Gatsby's integration with WordPress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants