X Tutup
Skip to content

fix(core): make .toRx() return Subject#4540

Closed
robwormald wants to merge 1 commit intoangular:masterfrom
robwormald:fix/eventemitter
Closed

fix(core): make .toRx() return Subject#4540
robwormald wants to merge 1 commit intoangular:masterfrom
robwormald:fix/eventemitter

Conversation

@robwormald
Copy link
Copy Markdown
Contributor

this was broken in the original great RxNext migration.

@PatrickJS
Copy link
Copy Markdown
Contributor

👍

this was broken in the original great RxNext migration.
fixes angular#4521
@rkirov
Copy link
Copy Markdown
Contributor

rkirov commented Oct 6, 2015

@jeffbcross @vsavkin can any one approve this. Sounds like a major bug fix.

@IgorMinar IgorMinar added this to the alpha-40 milestone Oct 6, 2015
@jeffbcross jeffbcross added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Oct 6, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #4540 on behalf of @jeffbcross to branch presubmit-jeffbcross-pr-4540.

@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Oct 6, 2015
@jeffbcross
Copy link
Copy Markdown
Contributor

Looks good to me for now.

@robwormald robwormald closed this in 4a36fd8 Oct 6, 2015
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

X Tutup