Skip to content

Fix for SSR2 fixture not working locally #24237

Merged
gaearon merged 5 commits into
react:mainfrom
kunalpanchal:main
Mar 31, 2022
Merged

Fix for SSR2 fixture not working locally #24237
gaearon merged 5 commits into
react:mainfrom
kunalpanchal:main

Conversation

@kunalpanchal

Copy link
Copy Markdown
Contributor

Summary

  • /fixtures/ssr2 lacked a README.md on how to set things up locally, so I have added a Readme file similar to that of /fixtures/ssr since the steps are the same.
  • Even after running the README steps I was not able to view the application in the browser and there were no error logs since the Options type does not have onCompleteShell.

How did you test this change?

yarn && yarn build # To build for the latest react

cd fixtures/ssr2
yarn && yarn install
yarn start # Works well ✅
yarn start:prod # Works well ✅

Comment thread fixtures/ssr2/server/render.js Outdated
@sizebot

sizebot commented Mar 31, 2022

Copy link
Copy Markdown

Comparing: 4db3ff6...3c2181c

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 131.30 kB 131.30 kB = 41.96 kB 41.96 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 136.36 kB 136.36 kB = 43.41 kB 43.41 kB
facebook-www/ReactDOM-prod.classic.js = 432.72 kB 432.72 kB = 79.59 kB 79.59 kB
facebook-www/ReactDOM-prod.modern.js = 417.72 kB 417.72 kB = 77.21 kB 77.21 kB
facebook-www/ReactDOMForked-prod.classic.js = 432.72 kB 432.72 kB = 79.59 kB 79.59 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 3c2181c

@gaearon
gaearon merged commit d352fd0 into react:main Mar 31, 2022
@gaearon

gaearon commented Mar 31, 2022

Copy link
Copy Markdown
Collaborator

thanks!

@kunalpanchal

Copy link
Copy Markdown
Contributor Author

Thank you @gaearon for the quick review. Btw, this is my first contribution to react 🎉

rickhanlonii pushed a commit that referenced this pull request Apr 13, 2022
* SS2 fixture not working locally fix

* Fix prettier issue

* prettier excess line fix

* Update render.js

* Update README.md

Co-authored-by: dan <[email protected]>
rickhanlonii pushed a commit that referenced this pull request Apr 14, 2022
* SS2 fixture not working locally fix

* Fix prettier issue

* prettier excess line fix

* Update render.js

* Update README.md

Co-authored-by: dan <[email protected]>
rickhanlonii pushed a commit that referenced this pull request Apr 14, 2022
* SS2 fixture not working locally fix

* Fix prettier issue

* prettier excess line fix

* Update render.js

* Update README.md

Co-authored-by: dan <[email protected]>
zhengjitf pushed a commit to zhengjitf/react that referenced this pull request Apr 15, 2022
* SS2 fixture not working locally fix

* Fix prettier issue

* prettier excess line fix

* Update render.js

* Update README.md

Co-authored-by: dan <[email protected]>
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
* SS2 fixture not working locally fix

* Fix prettier issue

* prettier excess line fix

* Update render.js

* Update README.md

Co-authored-by: dan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants