-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
chore(app): update to react 15.02 #1140
base: master
Are you sure you want to change the base?
Conversation
"react-router-redux": "^4.0.0", | ||
"redux": "^3.0.4", | ||
"redux-async-connect": "^1.0.0-rc2", | ||
"redux-connect": "^2.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
woooot!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
Pull request works fine in my repo! |
Nice work! Is the plan to get this merged sometime soon? |
Tested it, works fine. 👍 |
+1 this project really needs an update to latest dependencies... @medfreeman Thank You |
I've updated all dependencies and apply required changes on my fork, until this project revive.. @medfreeman Thank You |
why aren't pr merged ? |
desperately needed PR. please merge it. |
@erikras Can you please merge this pull request or update the dependencies to their newer versions? |
@sauravskumar99 This repository is in silent mode since Apr. Maybe we have to request a commit permission, or moving to a fork with someone maintains repository can be a option ... |
@hanjukim for the time being... @bertho-zero https://github.com/bertho-zero/react-redux-universal-hot-example has a fork with up to date dependecies... you may wanna checkout |
closes #1128