Skip to content

Conversation

@atscott
Copy link
Contributor

@atscott atscott commented Aug 5, 2024

…nes"

This commit updates the inside/outside NgZone detection of the hybrid CD scheduling to track the actual instance of the NgZone being used rather than the name "Angular" (how isInsideAngularZone works). This allows the scheduling to work correctly when there are multiple versions of Angular running on the page.

fixes #57261

@atscott atscott added area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release labels Aug 5, 2024
@ngbot ngbot bot added this to the Backlog milestone Aug 5, 2024
@atscott atscott force-pushed the isInAngularZoneFix branch 3 times, most recently from f45156b to 2c765d3 Compare August 5, 2024 23:17
@atscott atscott force-pushed the isInAngularZoneFix branch from 2c765d3 to fb1c077 Compare August 6, 2024 21:54
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Aug 6, 2024
@thePunderWoman thePunderWoman added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Aug 7, 2024
@atscott atscott force-pushed the isInAngularZoneFix branch from fb1c077 to c10b168 Compare August 7, 2024 17:17
@thePunderWoman thePunderWoman removed the action: merge The PR is ready for merge by the caretaker label Aug 7, 2024
…nes"

This commit updates the inside/outside NgZone detection of the hybrid CD
scheduling to track the actual instance of the NgZone being used rather
than the name "Angular" (how `isInsideAngularZone` works). This allows
the scheduling to work correctly when there are multiple versions of
Angular running on the page.

fixes angular#57261
@atscott atscott force-pushed the isInAngularZoneFix branch from c10b168 to 3062051 Compare August 7, 2024 17:47
@atscott atscott added action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews target: patch This PR is targeted for the next patch release labels Aug 7, 2024
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit 769b6e1.

The changes were merged into the following branches: main, 18.2.x

AndrewKushnir pushed a commit that referenced this pull request Aug 8, 2024
…nes" (#57267)

This commit updates the inside/outside NgZone detection of the hybrid CD
scheduling to track the actual instance of the NgZone being used rather
than the name "Angular" (how `isInsideAngularZone` works). This allows
the scheduling to work correctly when there are multiple versions of
Angular running on the page.

fixes #57261

PR Close #57267
@angular-automatic-lock-bot
Copy link

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 8, 2024
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: core Issues related to the framework runtime target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

zone based apps + new scheduler: signal changes in another zone not triggering CD

5 participants