X Tutup
Skip to content

fix(web worker): remove usages of deprecated zone API#5425

Closed
yjbanov wants to merge 1 commit intoangular:masterfrom
yjbanov:cleanup-overrideOnEventDone
Closed

fix(web worker): remove usages of deprecated zone API#5425
yjbanov wants to merge 1 commit intoangular:masterfrom
yjbanov:cleanup-overrideOnEventDone

Conversation

@yjbanov
Copy link
Copy Markdown
Contributor

@yjbanov yjbanov commented Nov 21, 2015

This replaces usages of overrideOnEventDone with onEventDone stream-based API

@yjbanov yjbanov force-pushed the cleanup-overrideOnEventDone branch 2 times, most recently from 9fbb48c to 5c177a8 Compare November 23, 2015 17:33
@yjbanov yjbanov force-pushed the cleanup-overrideOnEventDone branch from 5c177a8 to 16e5252 Compare November 24, 2015 18:32
@yjbanov yjbanov added the action: review The PR is still awaiting reviews from at least one requested reviewer label Nov 24, 2015
@vsavkin vsavkin added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 24, 2015
@vsavkin vsavkin assigned yjbanov and unassigned vsavkin Nov 24, 2015
@yjbanov yjbanov added action: merge The PR is ready for merge by the caretaker comp: dart-transformer and removed comp: dart-transformer labels Nov 25, 2015
@yjbanov
Copy link
Copy Markdown
Contributor Author

yjbanov commented Nov 25, 2015

@alxhub fyi, the build is actually green, Travis seems to fail to update the status for some reason.

@mary-poppins
Copy link
Copy Markdown

Merging PR #5425 on behalf of @vsavkin to branch presubmit-vsavkin-pr-5425.

@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

action: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup