-
Notifications
You must be signed in to change notification settings - Fork 175
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
Port to Redux #38
Comments
Imho rename and kee pthe flux in an old branch and keep a link in the readme.... Between redux and react-router 1.0 there is going to be A LOT of changes. Also, thanks for all the work you've been doing with redux lately! |
like @felipeleusin has said it, rename it and keep the old version. maybe as branch or maybe even only as link to current revision. |
agree with @felipeleusin. V. interested in seeing this port. |
It might be better to make a new repository redux-react-router-example |
Awesome, can't wait for this! |
The port will be part of the official Redux repo. |
The Redux port is here: https://github.com/rackt/redux/tree/master/examples/real-world |
I intend to port this example to Redux.
I'm not doing that to promote my library, but because reusable store utilities (like pagination) are way easier to implement as pure functions. What I have right now in
PaginatedStoreUtils
is a mess, and will be much more sensible with Redux approach.However, I understand that not everybody will be happy with this change, as Redux is not exactly Flux. I think it solves the same problems better than Flux, but that's my opinion and some may not share it.
Therefore I ask what you think:
To clarify again:
Thoughts?
The text was updated successfully, but these errors were encountered: