-
Notifications
You must be signed in to change notification settings - Fork 65
Update for react-15 #99
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
Conversation
fixes #95 |
This looks good to me. Have you been able to test it on any projects? @ethul Any thoughts on this? |
Yes, I did with https://github.com/coot/purescript-isomorphic-react-example Later I can push the testing branch. The react warning about 'createClass' is gone. |
I think this looks good. Thanks for the update!
…On Tue, Jun 27, 2017 at 01:54 Coot ***@***.***> wrote:
Yes, I did with
https://github.com/coot/purescript-isomorphic-react-example
Later I can push the testing branch. The react warning about 'createClass'
is gone.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#99 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAVYy-QFH7ksnQJRBJGicpPkzduv80o2ks5sIJkIgaJpZM4OEFv9>
.
|
Thanks! |
There is a typo in the release notes: it's an update for react-15, react-16 is still in development, though people are already using it. |
Fixed. Thanks! |
The bindings now require to install
create-react-class
alongside withreact
. This suppresses a warning in react-15. React-16 will not allow to useReact.createClass
and it's migrating towards react classes.