-
-
Notifications
You must be signed in to change notification settings - Fork 15.2k
Rename Actions to "Records"? #377
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
Comments
We've been bikeshedding on this for quite a while, and from the conversations I was in, I didn't find any better word than “actions”. A new term has to be significantly better to justify inventing yet another terminology. Redux names comes from “Flux with reducers” and it makes sense for me to keep the Flux terminology, at least to aid the beginners. In the new docs, we are calling them “actions” and “async actions”. I tried calling them “intents” but that just complicates docs because there is clearly a new term, but their role in Redux is supplementary rather than critical. |
Another good name I heard was Fact. Just recording for posterity. |
Events is a common term for facts in the event sourcing world. |
What about Additionally, I think |
I love the rename of Stores to Reducers, it really changes how you think about the code, and I noticed that there is some confusion between actual actions taken and the record of their results as Action objects.
Action objects are really only news items and they can be recorded by the reducers but they don't perform any work, that was already done. (started a request, got data from server etc).
Therefore I think it would help our mental model if they are renamed to Records (or NewsItems, Events, ...?).
Then ActionCreators can remain the "actions" they are colloquially (or even be renamed to just Actions).
It would look like this:
The text was updated successfully, but these errors were encountered: