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.

Sign up for GitHub

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

Jump to bottom

Move to Redux? #222

Closed
JustinTulloss opened this issue Jul 2, 2015 · 5 comments
Closed

Move to Redux? #222

JustinTulloss opened this issue Jul 2, 2015 · 5 comments

Comments

@JustinTulloss
Copy link
Contributor

Are there plans to move this library from Flummox to Redux? I personally would be more willing to add Redux as a dependency if this library (which I already depend on) used it.
@gaearon
Copy link
Member

gaearon commented Jul 5, 2015

I don't have plans at the moment, but eventually I'd do that.

@JustinTulloss
Copy link
Contributor Author

Sounds good!

@gaearon gaearon mentioned this issue Aug 7, 2015
Closed
@gaearon
Copy link
Member

gaearon commented Sep 15, 2015

Done in react-dnd@1.1.6 / dnd-core@1.2.0: react-dnd/dnd-core#21.

@ouchxp
Copy link

ouchxp commented Apr 22, 2016

@gaearon I suppose the documentation should be updated as well.
https://gaearon.github.io/react-dnd/docs-overview.html
The latest official document still saying it's backed by Flux.

@gaearon
Copy link
Member

gaearon commented Apr 22, 2016

Well, technically Redux is still rather similar to Flux. But a PR is welcome.

zargold added a commit to zargold/react-dnd that referenced this issue Aug 6, 2016
@zargold
adding Redux instead of or in addition to Flux in documentation per i…
6b5f1a6 
…ssue: react-dnd#222
kesne added a commit that referenced this issue Aug 18, 2016
@kesne
Merge pull request #501 from zargold/master
26e6beb 
Solves Issue #222 (documenting move to Redux)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@JustinTulloss @gaearon @ouchxp