Skip to content

Revert "feat: Monkey patches MessagePort onproperties (onmessage/onme…#35973

Closed
AndrewKushnir wants to merge 1 commit intoangular:masterfrom
AndrewKushnir:revert_34610
Closed

Revert "feat: Monkey patches MessagePort onproperties (onmessage/onme…#35973
AndrewKushnir wants to merge 1 commit intoangular:masterfrom
AndrewKushnir:revert_34610

Conversation

@AndrewKushnir
Copy link
Copy Markdown
Contributor

@AndrewKushnir AndrewKushnir commented Mar 9, 2020

…ssageerror) (#34610)"

This reverts commit 1882451.

Reason: breaks some g3 targets.

Note: reverting in master only, since original changes were merged only to master.

…ssageerror) (angular#34610)"

This reverts commit 1882451.

Reason: breaks some g3 targets.
@AndrewKushnir AndrewKushnir added the target: major This PR is targeted for the next major release label Mar 9, 2020
@pullapprove pullapprove Bot requested review from IgorMinar and JiaLiPassion March 9, 2020 20:52
@AndrewKushnir AndrewKushnir requested a review from matsko March 9, 2020 20:53
@AndrewKushnir AndrewKushnir added the area: zones Issues related to zone.js label Mar 9, 2020
@ngbot ngbot Bot added this to the needsTriage milestone Mar 9, 2020
@matsko matsko added the action: merge The PR is ready for merge by the caretaker label Mar 9, 2020
@matsko matsko closed this in 71309d2 Mar 9, 2020
@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 Apr 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: zones Issues related to zone.js cla: yes target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants