Skip to content

Conversation

@DanielSLew
Copy link
Contributor

Description

In order to unblock the gatsby-starter-shopify from being migrated to Gatsby v4 we needed to fix the gatsbyPath resolver. This resolver relies on extensions from the program, but this program wasn't getting fully set in the workers.

We also needed to use onPluginInit when compatible with the Gatsby version.

Documentation

Related Issues

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Sep 9, 2021
@DanielSLew DanielSLew changed the title Fix gatsby plugin page creator v4 fix(gatsby-plugin-page-creator): Fix gatsby plugin page creator v4 Sep 9, 2021
@DanielSLew DanielSLew added gatsby 4 and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Sep 9, 2021
@DanielSLew DanielSLew changed the title fix(gatsby-plugin-page-creator): Fix gatsby plugin page creator v4 feat(gatsby-plugin-page-creator): Fix gatsby plugin page creator v4 Sep 9, 2021
Copy link
Contributor

@wardpeet wardpeet left a comment

Choose a reason for hiding this comment

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

One small remark. I need to test it to make sure but looks good!

DanielSLew and others added 2 commits September 9, 2021 17:55
Co-authored-by: Ward Peeters <ward@coding-tech.com>
Copy link
Contributor

@wardpeet wardpeet left a comment

Choose a reason for hiding this comment

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

LGTM

@DanielSLew DanielSLew merged commit 4837b72 into master Sep 10, 2021
@DanielSLew DanielSLew deleted the fix-gatsby-plugin-page-creator-v4 branch September 10, 2021 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants