Skip to content

fix(gatsby): more reliable way to use prod versions of react/react-dom#29683

Merged
KyleAMathews merged 4 commits intorelease/2.32from
fix-dev-ssr-2
Feb 23, 2021
Merged

fix(gatsby): more reliable way to use prod versions of react/react-dom#29683
KyleAMathews merged 4 commits intorelease/2.32from
fix-dev-ssr-2

Conversation

@KyleAMathews
Copy link
Copy Markdown
Contributor

@KyleAMathews KyleAMathews commented Feb 22, 2021

back port of #29620 to v2

Fix #28138 (comment)

Set NODE_ENV to production in the child worker that does the SSR in dev.

back port of #29620 to v2

Fix #28138 (comment)

Set `NODE_ENV` to `production` in the child worker that does the SSR in dev.
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Feb 22, 2021
@KyleAMathews KyleAMathews added topic: ssr and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Feb 22, 2021
@KyleAMathews
Copy link
Copy Markdown
Contributor Author

Windows tests are failing on the node version used 🤔

@wardpeet
Copy link
Copy Markdown
Contributor

This seems fine, can you update snapshot (I think we already did this in master)

@pieh
Copy link
Copy Markdown
Contributor

pieh commented Feb 22, 2021

This seems fine, can you update snapshot (I think we already did this in master)

Other backport PRs also had this test failing because of that snapshot so I would say if we want to update snapshot for that branch, let's do this separately

@KyleAMathews
Copy link
Copy Markdown
Contributor Author

snapshot fix @ #29697

@KyleAMathews KyleAMathews merged commit 01d07b3 into release/2.32 Feb 23, 2021
@KyleAMathews KyleAMathews deleted the fix-dev-ssr-2 branch February 23, 2021 22:26
@ascorbic
Copy link
Copy Markdown
Contributor

Published in gatsby@2.32.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants