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

enhance: saga-effect take should be able to take multi actions; #1661

Merged
merged 1 commit into from Jun 1, 2018

Conversation

sjy
Copy link
Contributor

@sjy sjy commented May 2, 2018

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.9%) to 69.604% when pulling ae4653d on sjy:saga_take into e114cfb on dvajs:master.

@coveralls
Copy link

coveralls commented May 2, 2018

Coverage Status

Coverage increased (+0.2%) to 70.961% when pulling 2d72ff6 on sjy:saga_take into a6f29f1 on dvajs:master.

@sorrycc
Copy link
Member

sorrycc commented May 21, 2018

加一个额外的 take with array 的用例吧。

@sjy
Copy link
Contributor Author

sjy commented May 28, 2018

@sorrycc 加了新的ut,revert对已有ut的修改

@sorrycc sorrycc merged commit e8fdac3 into dvajs:master Jun 1, 2018
@sjy sjy deleted the saga_take branch June 1, 2018 07:35
GoToBoy pushed a commit to GoToBoy/dva that referenced this pull request Oct 12, 2023
enhance: saga-effect take should be able to take multi actions;
GoToBoy pushed a commit to GoToBoy/dva that referenced this pull request Oct 12, 2023
enhance: saga-effect take should be able to take multi actions;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants