Skip to content

fix(repeatWhen): wait for source to complete after notifier completes#2209

Merged
benlesh merged 1 commit intoReactiveX:masterfrom
mpodlasin:repeat-when
Jan 29, 2017
Merged

fix(repeatWhen): wait for source to complete after notifier completes#2209
benlesh merged 1 commit intoReactiveX:masterfrom
mpodlasin:repeat-when

Conversation

@mpodlasin
Copy link
Copy Markdown
Contributor

Description:

After notifier completes wait for source observable to complete instead
of ending stream immediately

Related issue (if exists):

Closes #2054

After notifier completes wait for source observable to complete instead
of ending stream immediately

Closes #2054
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 22, 2016

Coverage Status

Coverage increased (+0.0007%) to 97.674% when pulling e37edf0 on Podlas29:repeat-when into 45b842c on ReactiveX:master.

@benlesh
Copy link
Copy Markdown
Member

benlesh commented Jan 4, 2017

@kwonoj or @mattpodwysocki can we confirm this is the expected behavior from Rx4?

@benlesh
Copy link
Copy Markdown
Member

benlesh commented Jan 29, 2017

I've verified this is the expected behavior from Rx4.. We meant to target that, so this is indeed a bug fix.

@benlesh benlesh merged commit c65a098 into ReactiveX:master Jan 29, 2017
@lock
Copy link
Copy Markdown

lock Bot commented Jun 6, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Jun 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants