-
-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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-router-redux throws warnings with react 15.5 #4932
Comments
Sorry to nit, but these warnings do not stem from |
Should be fixed on master. I'll see about getting a release out soon. |
@timdorr this issue is regarding RR3 (which also uses the now deprecated If you accept PR's for the v3 branch I'd be happy to contribute. |
Yep, we do. I will take care of it now, though. It's not that hard to go through, luckily. |
@simonkberg I noticed that the error was due to the react-router version that's why I mentioned it in the report 😉 @timdorr thanks for the speedy response and fix 👍 |
@timdorr I met the same warning with [email protected], so whether you fixing it in 2.x version? |
No, we're not maintaining the 2.x versions. Upgrade to 3.x, it's just 2.x without the deprecation warnings. |
Is this change still waiting to get into the v3 branch? |
@FoxxMD the fixes has been released https://github.com/ReactTraining/react-router/releases/tag/v3.0.4 |
@simonkberg thanks! I didn't see it on the release page but I didn't scroll down far enough :) |
@simonkberg I upgraded to |
@kevinguard are you sure they're coming from RR? could you post the full stack trace? |
I just upgraded to |
Version
react-router 3.0.3
react-router-redux 4.0.8
Issue
The changes in react 15.5 regarding PropTypes and createClass make react-router-redux throw two warnings.
Will this issue be solved or is it something that will be addressed when react-router-redux 5 is released with support for react-router 4?
The text was updated successfully, but these errors were encountered: