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.

Sign up for GitHub

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

Jump to bottom

fix(mergeAll): add source subscription to composite before actually subscribing #2479

Merged
merged 1 commit into from
Jul 26, 2018
Merged

fix(mergeAll): add source subscription to composite before actually subscribing #2479

merged 1 commit into from
Jul 26, 2018

Commits on Jul 7, 2018

  1. fix(mergeAll): add source subscription to composite before actually s… 

    …ubscribing

Add subscriptions for source Observables to mergeAll composite subscription
before actually subscribing to any of these Observables, so that if
source Observable emits synchronously and consumer of mergeAll unsubscribes
at that moment (for example `take` operator), subscription to source is
unsubscribed as well and Observable stops emitting.

Closes #2476
    @mpodlasin @cartant
    mpodlasin authored and cartant committed Jul 7, 2018
    Configuration menu
    Copy the full SHA
    f5970bf View commit details
    Browse the repository at this point in the history