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

Operator: switchIfEmpty #2585

Merged
merged 5 commits into from
Feb 3, 2015
Merged

Operator: switchIfEmpty #2585

merged 5 commits into from
Feb 3, 2015

Conversation

akarnokd
Copy link
Member

@akarnokd akarnokd commented Feb 3, 2015

Changes to #2091

Alex Wenckus and others added 4 commits December 19, 2014 15:13
…nd emits the items in an Observable if the source is empty
Ensure that we unsubscribe upstream "parent" when switching to alternate. That way upstream will trigger unsubscribe when the first Observable completes. Added test.
Child should contain downstream subscriptions - not parent.
@akarnokd akarnokd changed the title Switch if empty Operator: switchIfEmpty Feb 3, 2015
akarnokd added a commit that referenced this pull request Feb 3, 2015
@akarnokd akarnokd merged commit e934c81 into ReactiveX:1.x Feb 3, 2015
@akarnokd akarnokd deleted the switch_if_empty branch February 3, 2015 08:48
@alexwen
Copy link

alexwen commented Feb 3, 2015

thanks @akarnokd for fixing up and merging!

@benjchristensen benjchristensen mentioned this pull request Feb 3, 2015
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.

2 participants