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 15 #47

Merged
merged 2 commits into from
Nov 22, 2016
Merged

React 15 #47

merged 2 commits into from
Nov 22, 2016

Conversation

eatyourgreens
Copy link
Contributor

Upgrades the React packages to 15.3 and removes defaultValue from the editor.

React 15 enforces using defaultValue or value, but not both, on form controls.
@eatyourgreens
Copy link
Contributor Author

This should be merged as part of zooniverse/Panoptes-Front-End#2984 to keep the React versions in synch.

@srallen
Copy link
Contributor

srallen commented Nov 11, 2016

Cool. I'm taking a look at both PRs this morning.

@rogerhutchings
Copy link
Contributor

LGTM - I'll leave the PR open, as it'll need a version bump as well

@rogerhutchings
Copy link
Contributor

Merging now, as there's a version bump as part of #48

@rogerhutchings rogerhutchings merged commit d9aca86 into master Nov 22, 2016
@rogerhutchings rogerhutchings deleted the react-15 branch November 22, 2016 15:11
@eatyourgreens
Copy link
Contributor Author

@rogerhutchings rogerhutchings restored the react-15 branch November 22, 2016 15:18
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.

3 participants