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

rearrange action_creators #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

taddy569
Copy link
Contributor

Hello, I've just swapped these two action creators, based on:

  • Before: "the first one that activates the side-effect to fetch stories by a search term and the second one that adds the fetched stories to your Redux store."

  • After: "only the second action needs to be intercepted in your storyReducer to store the stories. The first action is only used to activate the saga in your root saga."

Hope that I can help! Thank you for reading this.

@rwieruch
Copy link
Member

Nothing is changed in the code, is there?

@taddy569
Copy link
Contributor Author

Yes, nothing is changed in the code.
I've swapped "doFetchStories" as the first action creator - activates the side-effect to fetch stories and "doAddStories" as second creator - need to intercepted in reducer.

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.

None yet

2 participants