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

Update react-modal to work with React v0.14 and fix deprectated warnings. #68

Closed
wants to merge 1 commit into from

Conversation

rogeliog
Copy link

Fixes #67

It might be a good idea to wait until React v0.14.0 gets released or maybe we can merge it in react-0.14 branch

😄

@mzabriskie
Copy link
Member

@rogeliog Thank you for the PR! Let's wait until 0.14.0 is released. In the mean time #65 will at least let you use the RC.

@@ -44,7 +47,7 @@
"element-class": "^0.2.0"
},
"peerDependencies": {
"react": ">=0.13.3"
"react": ">=0.14.0.rc1"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rogeliog Thanks for your great work. I'm currently using your fork now but getting an error on npm install because of this. I guess it should be -rc1 instead of .rc1.

@claydiffrient
Copy link
Contributor

@rogeliog Looks like React 0.14 support was already merged in as of version 0.6.0. I'm going to go ahead and close this one out to clean things up.

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.

4 participants