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

revert Gatsby 5 changes #959

Merged
merged 50 commits into from
Dec 6, 2023
Merged

revert Gatsby 5 changes #959

merged 50 commits into from
Dec 6, 2023

Conversation

sunnyzanchi
Copy link
Contributor

@sunnyzanchi sunnyzanchi commented Dec 4, 2023

this is a lot of changed files, but primarily it just reverts a bunch of commits so that the codebase is back to its state @ 7.1.3, before we made any changes for Gatsby 5. there are some non-Gatsby 5 changes — like removing the AnimatedCard component — that i left in

testing

i'd recommend clearing node_modules first, then yarning and yarn starting to test the site. the best way to review this PR is probably not to look at the commits or files changed, but to look at the diff between this branch and the v7.1.3 tag. you can do this after checking out this branch with yarn diff v7.1.3. the changelog is huge, you can exclude it with git diff v7.1.3 ':!*.md'

things to look out for in the diff:

  • any import/require changes, especially anything using .mjs
  • any dependency changes in package.json, demo/package.json, or packages/gatsby-theme-newrelic/package.json
  • any GraphQL changes

This reverts commit 60db122.

fix: readd `gatsby-plugin-portal` as a dependency
i couldn't figure out why this wasn't reverting how i expected it to,
so i just copied the code from v7.1.3 and replaced it here
This reverts commit 9bac64c.
BREAKING CHANGE: reverts the Gatsby 5 changes so the theme
is back on Gatsby 4.
@sunnyzanchi sunnyzanchi marked this pull request as ready for review December 6, 2023 19:28
@sunnyzanchi sunnyzanchi requested review from a team, clarkmcadoo and LizBaker and removed request for a team December 6, 2023 19:28
Copy link
Contributor

@tabathadelane tabathadelane left a comment

Choose a reason for hiding this comment

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

No issues stand out. The demo js and mdx pages are working

@sunnyzanchi sunnyzanchi merged commit 4271a27 into develop Dec 6, 2023
3 checks passed
@sunnyzanchi sunnyzanchi deleted the sunny/revert-gatsby5 branch December 6, 2023 21:48
sunnyzanchi added a commit that referenced this pull request Dec 6, 2023
feat!: revert Gatsby 5 changes (#959)
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