-
Notifications
You must be signed in to change notification settings - Fork 97
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
BufferWhileBy should flush source stream on falsy events from "otherObs" #116
Labels
Comments
rpominov
added a commit
that referenced
this issue
Jun 26, 2015
I've added an option for this. Will it solve your problem? Also can you try it out before we release (it in master now)? |
Here's my implementation where it is flushed this way by default. I'll try your version in few minutes, thank you! |
Yes, it works the same way as I expected with even fewer changes, thank you once again! :) |
rpominov
added a commit
that referenced
this issue
Jun 27, 2015
* master: cleanup repository after release 2.7.0 update changelog add `flushOnChange` option to `a.bufferWhileBy` (rel #116) update deps update changelog update changelog
Released in |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Instead of flushing on the next event from the source stream. So it should rather be like:
The text was updated successfully, but these errors were encountered: