-
Notifications
You must be signed in to change notification settings - Fork 755
Update to React 0.14 #365
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
Update to React 0.14 #365
Conversation
How are the ones listed on https://facebook.github.io/react/downloads.html generated? |
Score! It looks like that includes |
😩 It includes |
I think we can ship with react-rails.js and that should henceforth include whatever is needed to get react running.
And in Application -
|
When 0.14 React will be available in this gem ? I developoing rails server, but i dont want to use outdated react version. |
@rmosolgo Thanks for your effort on getting React 0.14 in place. While I'm (relatively) new to React and this gem if you need help getting this sorted out I'd be happy to dive in. |
@onpaws I'm hoping to take a pass at this on Sunday or Monday, but if you're interested, definitely take a shot at it! Here's what I've got in mind: Goals
MethodI'm open to anything that accomplishes the goals above. I haven't tried it yet, but this is what I'd try:
Not sure about
|
Awesome! Your plan sounds great. Regarding your last item, I noticed angularjs-rails appears to offer something similar to what you describe. Perhaps this might be helpful? |
I took a pass at this here: #376 |
This is kind of a non-pull request. I've used our existing update process, but it doesn't work 😞
The bower package doesn't include
ReactDOMServer
, which we need for server rendering.Also, it's kind of a downer to introduce that second require:
Because this is Ruby on Rails, I think it would be acceptable to assume that we want
react-dom
.So, maybe we need a new update & build process:
react
&react-dom
,react-dom/server
?)Any suggestions?