You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm currently trying out this package and noticed there is no stubFork in the package. Taking the blind man approach I copied most of the stubCall code and made some minor adjustments to make it work (thus far) for my needs.
So far I haven't encountered any problems, but I also didn't give the implementation much thought. Given that there was already some code related to forks (e.g. forkGeneratorFn) it makes me wonder why stubFork wasn't present. Care to shed a light? Was it just not considered? Perhaps a time constraint, or are there known issues/limitations with the simplistic approach I've taken?
Hope to hear from you.
The text was updated successfully, but these errors were encountered:
stubFork was not considered but should be easy to add it, you can submit a pull request. The actual code related to forks is to traverse the tree of effects generated by a fork with a generator function.
I don't think my (PR)[https://github.com//pull/2] does what you have in mind. I basically use it like the stubCall where I can return my own value. e.g. saga
yield fork(walletUseFlow, params)
const action = yield take([actionMsg.wallet.LOGOUT, actionMsg.wallet.LOGIN_ERROR])
if (action.type == actionMsg.wallet.LOGOUT) { // do something }
Hey,
I'm currently trying out this package and noticed there is no
stubFork
in the package. Taking the blind man approach I copied most of thestubCall
code and made some minor adjustments to make it work (thus far) for my needs.So far I haven't encountered any problems, but I also didn't give the implementation much thought. Given that there was already some code related to forks (e.g.
forkGeneratorFn
) it makes me wonder whystubFork
wasn't present. Care to shed a light? Was it just not considered? Perhaps a time constraint, or are there known issues/limitations with the simplistic approach I've taken?Hope to hear from you.
The text was updated successfully, but these errors were encountered: