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

DOP-4406: Fixed React minified error in Suspense #1032

Merged
merged 3 commits into from
Mar 5, 2024
Merged

Conversation

mmeigs
Copy link
Collaborator

@mmeigs mmeigs commented Mar 5, 2024

Stories/Links:

DOP-4406

Current Behavior:

Landing
Cloud-docs

Staging Links:

Landing
Cloud-docs

Notes:

In the current prod site, you can see the react minified error #419 which is here.

This happens because of SSR hydration differences when using React.Suspense and Gatsby. There are a couple open issues with Gatsby for this (even one with our friend Grayson), but this PR introduces a very simple fix to allow the SSR to compile the static html with the fallback and allow stable hydration. Tricks on tricks. I would assume Gatsby will not be fixing this anytime soon.

README updates

    • This PR introduces changes that should be reflected in the README, and I have made those updates.
    • This PR does not introduce changes that should be reflected in the README

Copy link
Collaborator

@branberry branberry left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@mmeigs mmeigs merged commit 7539256 into main Mar 5, 2024
4 checks passed
@mmeigs mmeigs deleted the DOP-4406-suspense branch March 5, 2024 20:56
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