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

Server rendering example #45

Closed
wants to merge 2 commits into from
Closed

Conversation

justin808
Copy link
Member

WIP right now.

* TEMP COMMIT for pairing...
* Path for react-rails is ~/a/react-rails

TIME TO HACK!

commit 7cacc8a5206d22cd480b657bb85b47bb45e02e04
Merge: e8ced3b 88dad94
Author: Justin Gordon <[email protected]>
Date:   Mon Jul 13 18:39:37 2015 -1000

    Merge branch 'server-rendering' of https://github.com/nikhilbaradwaj/react-webpack-rails-tutorial into nikhilbaradwaj-server-rendering

    * 'server-rendering' of https://github.com/nikhilbaradwaj/react-webpack-rails-tutorial:
      capybara-webkit was removed by mistake because I could not install this gem for some reason.
      Minor changes to App.jsx
      React-rails gem and setup for server side rendering.

commit 5572adf
Author: Nikhil Baradwaj <[email protected]>
Date:   Mon Jul 13 21:56:57 2015 -0400

    React-rails gem and setup for server side rendering.
* not working
* getting error on the document thing
* Need to make sure that webpack global export is done correctly
* If we export the App component globally, then that should not conflict
  with anything.
* Once we can export the App component, then we can pass in the comments
  as JSON, so it server renders.
@justin808
Copy link
Member Author

See #59. Server rendering will be included with that change.

@justin808
Copy link
Member Author

Not going to do it this way. See https://github.com/shakacode/react_rails_server_rendering/

@justin808 justin808 closed this Sep 5, 2015
@robwise robwise deleted the server-rendering-example branch October 22, 2015 04:36
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.

None yet

1 participant