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

Redux #173

Closed
cmnstmntmn opened this issue Jul 17, 2015 · 19 comments
Closed

Redux #173

cmnstmntmn opened this issue Jul 17, 2015 · 19 comments

Comments

@cmnstmntmn
Copy link

cmnstmntmn commented Jul 17, 2015

redux as a flux alternative

Watch Dan Abramov's talk on Hot Reloading with Time Travel.

Ref #579

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@timdoes
Copy link

timdoes commented Jul 21, 2015

+1

1 similar comment
@tanertopal
Copy link

👍

@Hyra
Copy link

Hyra commented Aug 26, 2015

😸

@mirchiseth
Copy link

+1

4 similar comments
@nurrony
Copy link

nurrony commented Sep 22, 2015

+1

@vkammerer
Copy link

+1

@sebastianpicklum
Copy link

+1

@dvic
Copy link

dvic commented Jan 29, 2016

+1

@rustanacexd
Copy link

+1 yes please

@dzcpy
Copy link

dzcpy commented Feb 2, 2016

+1

@tconroy
Copy link

tconroy commented Feb 2, 2016

👍 !!!

@learntoswim
Copy link

+1

1 similar comment
@buildbreakdo
Copy link

+1

@rusenask
Copy link

could we get some kind of discussion here instead of "+1" ? I am also for redux as it looks like a better and cleaner approach although it would be interesting to hear other opinions.

@tconroy
Copy link

tconroy commented Mar 24, 2016

@rusenask I mean... that pretty much sums it up. It's a cleaner flux architecture pattern, and rapidly becoming the community favorite implementation. We could fork this starter kit into two variants (a Flux and a Redux version) if the maintainer isn't interested in swapping out for Redux, but I think it would be in the best interest of the community to do so.

@langpavel
Copy link
Collaborator

@koistya I'm interesting in implementing redux in this project. Will be PR possibly accepted? I'm preparing another PR for react-intl support but I need some state sharing for selected language and other react-intl configuration

@koistya
Copy link
Member

koistya commented Apr 14, 2016

@langpavel yes, it can be merged into feature/redux branch for now. The problem is that many developer prefer Relay over Redux. I need to conduct a poll to check how many RSK users would like to see Redux on the master branch, how many Relay, and how many would like to keep it Flux-agnostic.

@langpavel
Copy link
Collaborator

@koistya I'm interesting in Relay too. I thing that can be possible and beneficial to have both but this will need some more digging

langpavel added a commit to langpavel/react-starter-kit that referenced this issue Apr 26, 2016
langpavel added a commit to langpavel/react-starter-kit that referenced this issue Apr 26, 2016
@koistya koistya closed this as completed in ba597af May 3, 2016
@koistya
Copy link
Member

koistya commented Sep 27, 2016

A bare minimum Redux integration (example): #883

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

No branches or pull requests