Skip to content

docs(recipes): add ssr recipe [fixes #54]#99

Merged
shellscape merged 1 commit intoshellscape:masterfrom
smashercosmo:react-ssr
Jan 29, 2019
Merged

docs(recipes): add ssr recipe [fixes #54]#99
shellscape merged 1 commit intoshellscape:masterfrom
smashercosmo:react-ssr

Conversation

@smashercosmo
Copy link
Contributor

This PR contains:

  • bugfix
  • feature
  • refactor
  • tests
  • documentation
  • metadata

Breaking Changes?

  • yes
  • no

Please Describe Your Changes

SSR example added [fixes #54]

Copy link
Owner

@shellscape shellscape left a comment

Choose a reason for hiding this comment

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

🎉 Thanks so much for doing this. Looks good to me, though I am no React expert. I will let the others chime in before merging.

@smashercosmo
Copy link
Contributor Author

TODO:

  1. Better README
  2. To make DX even better webpack-plugin-serve should be able to suspend until bundle is ready. With webpack-dev-middleware it was possible with 'serverRenderer' option.

@shellscape
Copy link
Owner

To make DX even better webpack-plugin-serve should be able to suspend until bundle is ready. With webpack-dev-middleware it was possible with 'serverRenderer' option.

If you have the time, please open a new feature issue, and we can work on making that happen!

Copy link
Collaborator

@matheus1lva matheus1lva left a comment

Choose a reason for hiding this comment

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

LGTM too, and thank you for the effort!

@sibelius can you double check? You have more experience on SSR i guess.

@shellscape
Copy link
Owner

OK @sibelius has given us the thumbs up offline. thanks @smashercosmo

@shellscape shellscape merged commit 74af036 into shellscape:master Jan 29, 2019
@smashercosmo
Copy link
Contributor Author

Awesome) Thx for amazing tool.

@shellscape shellscape mentioned this pull request Mar 19, 2019
18 tasks
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.

recipe: server-side rendering

3 participants