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

react-rails 2.0 #448

Closed
17 tasks done
rmosolgo opened this issue Jan 22, 2016 · 13 comments
Closed
17 tasks done

react-rails 2.0 #448

rmosolgo opened this issue Jan 22, 2016 · 13 comments
Milestone

Comments

@rmosolgo
Copy link
Member

rmosolgo commented Jan 22, 2016

There are some issues building up, I think they will need some breaking changes. Here's what's on my list. Do you have anything else that's missing?

@twe4ked
Copy link

twe4ked commented Jan 23, 2016

Thanks for creating this issue @rmosolgo. Looks like #450 could be a dot-point under Support Rails 5.0.

Edit: Just realised #450 is a duplicate of #404. Sorry.

@rmosolgo
Copy link
Member Author

Thanks, I added a note!

@SylarRuby
Copy link

I've just seen this list for version 2. Ok, support for rails 5 as I'm getting this: #467

@connorshea
Copy link

Do you mean Sprockets 3 or Sprockets 4? At the moment I'm testing Turbolinks 5 and Sprockets 4.0.0.beta2 and react-rails isn't playing well with one or both of them.

@rmosolgo
Copy link
Member Author

rmosolgo commented Mar 3, 2016

Oh, I didn't know Sprockets 4 was so close to launching! Will it be part of Rails 5?

I expect Turbolinks 5 to be supported in the latest (1.6.2). Please open an issue if you have any trouble!

@connorshea
Copy link

@rmosolgo Upgrading from 3.x and 4.x: https://github.com/rails/sprockets/blob/master/UPGRADING.md

I don't think it'll be apart of Rails 5, at least not as far as I know.

@connorshea
Copy link

@rmosolgo and thank you very very much for maintaining the gem for so long and so well! I was so happy to see react-rails had already been upgraded to support Rails 5, most other gems were throwing deprecations warnings right and left.

@connorshea
Copy link

@rmosolgo update: It looks like this was entirely a Turbolinks 5 problem. Since Turbolinks 5 has no guide for updating, I didn't realize the events had changed, so all our JavaScript stopped loading.

Using Turbolinks Classic and Sprockets 5.0.0 works fine with react-rails, apologies!

@connorshea
Copy link

@rmosolgo you can probably remove the renderToString TODO, React v15.0 will remove it.

@rmosolgo
Copy link
Member Author

rmosolgo commented Mar 8, 2016

They're still on ReactDOMServer though, right? I think we still need them somehow, that's how we implement server-side rendering!

For my part, I need renderToString in the browser for integrating with some other libraries, so I'd like to have a nice way to get it from react-rails!

@connorshea
Copy link

Not sure, I guess you'd have to test 15.0-rc1 to see what happens.

@drush
Copy link

drush commented Apr 22, 2016

Is React 15.0 support implicit for this issue, or should it be added to the checklist? Don't see a 15.0 ticket elsewhere.

@rmosolgo
Copy link
Member Author

@drush React 15 was merged in #512 and just shipped in 1.7.0 :)

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

No branches or pull requests

5 participants