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

Pass dispatch to higher-order middleware and re-implement thunk #120

Closed
wants to merge 1 commit into from

Conversation

acdlite
Copy link
Collaborator

@acdlite acdlite commented Jun 17, 2015

See #113 (comment) for context.


return recurse;
};
export default function thunkMiddleware(getState, dispatch) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, so if the thunk ain't the first one, dispatch will re-start the chain right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep!

This was referenced Jun 22, 2015
@gaearon
Copy link
Contributor

gaearon commented Jun 30, 2015

Implemented in #195. (Sorry, it was a pain to merge so I just copied the idea.)

@gaearon gaearon closed this Jun 30, 2015
@gaearon gaearon deleted the pass-dispatch-to-middleware branch June 30, 2015 21:46
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