-
Notifications
You must be signed in to change notification settings - Fork 54
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
Implement Flux Standard Actions? #23
Comments
Maybe, what is the benefit? Interoperability? One thing I don't agree is using the same action for the the event, the success and the error. Most likely if we use this we would keep using separate actions e.g. 'ON_FETCH_ERROR'. |
The benefit would be standardization. As it is now, when dealing with I imagine |
@orbiteleven do you mean to just make
I like the second idea, because this three actions is the one, but it's a pain to monitor them. |
@sporto this would defeat the point imo, I wouldn't mind seeing a configuration option to allow for FSA |
Any interest in implementing Flux Standard Actions? https://github.com/acdlite/flux-standard-action
Selfishly this makes sense as I blend both redux-actions (https://github.com/acdlite/redux-actions) and redux-crud in my applications and would like a consistent action interface.
Understandably this would be a breaking change, but IMO worth it... if there's interest.
The text was updated successfully, but these errors were encountered: