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

Add support for Static and Streaming SSR #18

Merged
merged 3 commits into from
Dec 25, 2017
Merged

Add support for Static and Streaming SSR #18

merged 3 commits into from
Dec 25, 2017

Conversation

tizmagik
Copy link
Owner

h/t @jamesjjk for kicking off this work in #17

fixes #14
fixes #13

jamesjjk and others added 3 commits December 25, 2017 08:24
* Add static sir support

* Postinstall git support

* Build only

* Enabled src files

* Bug fix buildselector

* Babel post install bug

* Fixed build selector

* Fix strings

* Update react head
@tizmagik tizmagik merged commit a84480f into master Dec 25, 2017
@tizmagik tizmagik deleted the static-ssr branch December 25, 2017 14:05
zhangciwu pushed a commit to zhangciwu/react-head that referenced this pull request Aug 10, 2018
* Feature staticssr support + escaping string (tizmagik#17)

* Add static sir support

* Postinstall git support

* Build only

* Enabled src files

* Bug fix buildselector

* Babel post install bug

* Fixed build selector

* Fix strings

* Update react head

* Remove build folder, leave off escape, add data-rh

* Remove build
@oyeanuj
Copy link

oyeanuj commented Dec 28, 2018

@tizmagik @jamesjjk Is there any example or docs on integrating react-head with streaming SSR since it is such a different beast from static markup?

@jamesjjk
Copy link

@oyeanuj We used a proprietary approach as this can be a fairly complex beast depending on what you want to achieve. You may want to decide to do this synchronously (blocking) and use secondary thread to deal with you dynamic content and render order respectively. Happy to comment on a more detailed issue.

@oyeanuj
Copy link

oyeanuj commented Dec 28, 2018

@jamesjjk here is the detailed issue: #74

looking forward to hearing your thoughts!

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.

Server rendering methods other than renderToString(headTags) renderToNodeStream support
3 participants