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 branch #67

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Redux branch #67

wants to merge 2 commits into from

Conversation

ShockiTV
Copy link
Contributor

Working redux example (tests not updated yet).
Let's correct me if files are not placed correctly or against mantra specs.

@kokjinsam
Copy link

@ShockiTV, thanks for doing this! I'm learning Redux at the moment. How's your experience with Redux? Is it better than using ReactiveDict? ReactiveDict-based localstate gets a little messy for me now. I'm looking to switch to redux. Would love to hear about your experience first before I switch.

@ShockiTV
Copy link
Contributor Author

It is my first redux experience :D
Still I am little confused form this, as mantra actions are quite complex.
And in Redux naming Actions are something else.
But if I put only redux Actions as Mantra actions, there would have to be very ugly high level reducers calling smaller reducers.
I did not feel very good with that.
Let's wait what maintainers will think :D

@swennemans
Copy link

@ShockiTV regarding actions/reducers etc. You could take a look at using the redux-ducks concept > https://github.com/erikras/ducks-modular-redux

@swennemans
Copy link

Maybe also relevant: kadirahq/mantra#103

I'm also looking for the correct way to integrate Redux with MantraJS. Maybe we can learn something from each other.

@ghost
Copy link

ghost commented Mar 14, 2016

I've been working on a boilerplate (work in progress) using React-Router, Redux, React-Router-Redux, and React Helmet, take a look here. It has a working example. Feedback welcome.

@swennemans
Copy link

@cheersjosh I'm looking forward to a more comprehensive example, using containers, data fetching etc. I'm happy to discuss it :)

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

Successfully merging this pull request may close these issues.

3 participants