Skip to content

Update for react-15#99

Merged
paf31 merged 6 commits into
purescript-contrib:masterfrom
coot:react-15
Jun 27, 2017
Merged

Update for react-15#99
paf31 merged 6 commits into
purescript-contrib:masterfrom
coot:react-15

Conversation

@coot

@coot coot commented Jun 23, 2017

Copy link
Copy Markdown

The bindings now require to install create-react-class alongside with react. This suppresses a warning in react-15. React-16 will not allow to use React.createClass and it's migrating towards react classes.

@coot

coot commented Jun 23, 2017

Copy link
Copy Markdown
Author

fixes #95

@paf31

paf31 commented Jun 27, 2017

Copy link
Copy Markdown
Contributor

This looks good to me. Have you been able to test it on any projects?

@ethul Any thoughts on this?

@coot

coot commented Jun 27, 2017

Copy link
Copy Markdown
Author

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.

@ethul

ethul commented Jun 27, 2017 via email

Copy link
Copy Markdown
Contributor

@paf31 paf31 merged commit f753b9f into purescript-contrib:master Jun 27, 2017
@paf31

paf31 commented Jun 27, 2017

Copy link
Copy Markdown
Contributor

Thanks!

@coot coot deleted the react-15 branch June 27, 2017 21:54
@coot

coot commented Jun 28, 2017

Copy link
Copy Markdown
Author

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.
https://github.com/purescript-contrib/purescript-react/releases/tag/v4.0.0

@ethul

ethul commented Jun 28, 2017

Copy link
Copy Markdown
Contributor

Fixed. Thanks!

@paulyoung paulyoung mentioned this pull request Aug 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants