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

chore(Demos): add SSR support for all react-live demos #1425

Merged
merged 1 commit into from
May 31, 2022

Conversation

tujoworker
Copy link
Member

@tujoworker tujoworker commented May 31, 2022

The advantages of SSR (SSG) support is that the build of the portal will run all the code examples – and if there is a JavaScript/React issue, the build will break.

We use react-live for showing demos/examples. It used to support SSR before v2.3.0 (a year ago) but missed it when they converted it to React Hooks. But they want to support it like this issue shows. So I made a PR. But until they eventually release it, I made a quick build of it and use it directly inside our repo.

@codesandbox-ci
Copy link

codesandbox-ci bot commented May 31, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit cdf5bc2:

Sandbox Source
eufemia-starter Configuration

@gatsby-cloud
Copy link

gatsby-cloud bot commented May 31, 2022

Gatsby Cloud Build Report

DNB Eufemia Portal

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 11m

Performance

Lighthouse report

Metric Score
Performance 🔶 67
Accessibility 💚 100
Best Practices 💚 100
SEO 💚 92

🔗 View full report

@tujoworker tujoworker requested a review from langz May 31, 2022 09:04
@tujoworker tujoworker marked this pull request as ready for review May 31, 2022 09:04
@tujoworker tujoworker changed the title chore(Portal): enhance ssr on examples/demos chore(Demos): add SSR support for all react-live demos May 31, 2022
@tujoworker tujoworker merged commit d1da715 into main May 31, 2022
@tujoworker tujoworker deleted the chore/react-live-ssr branch May 31, 2022 11:29
@tujoworker
Copy link
Member Author

🎉 This PR is included in version 9.27.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants