Skip to content

Prevent extra change detections in zoneless scheduler with manual calls to ApplicationRef.tick - #55290

Closed
atscott wants to merge 3 commits into
angular:mainfrom
atscott:appreftick
Closed

atscott wants to merge 3 commits into
angular:mainfrom
atscott:appreftick

Conversation

@atscott

@atscott atscott commented Apr 10, 2024

Copy link
Copy Markdown
Contributor

See individual commits

@atscott atscott added area: core Issues related to the framework runtime target: major This PR is targeted for the next major release labels Apr 10, 2024
@ngbot ngbot Bot added this to the Backlog milestone Apr 10, 2024
@atscott
atscott force-pushed the appreftick branch 3 times, most recently from 2773c28 to c70641a Compare April 11, 2024 02:10
@atscott
atscott requested a review from alxhub April 11, 2024 22:08
atscott added 2 commits April 12, 2024 16:35
…f.tick

This commit ensures that change detection is not scheduled if the
scheduler is notified during an applicationRef.tick
…hange detections

This commit ensures that manually calling ApplicationRef.tick will
result in any scheduled change detections being canceled. There is no
need for the scheduled one to run because it was manually done by the
`tick` already.
@atscott
atscott force-pushed the appreftick branch 2 times, most recently from c19667d to 9266108 Compare April 15, 2024 16:17
Comment thread packages/core/src/change_detection/scheduling/ng_zone_scheduling.ts Outdated
Comment thread packages/core/src/change_detection/scheduling/zoneless_scheduling_impl.ts Outdated
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Apr 15, 2024
@atscott atscott added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Apr 16, 2024
@atscott

atscott commented Apr 16, 2024

Copy link
Copy Markdown
Contributor Author

merge assistance: will need to patch messages custom implementation from cl/625061775

@pkozlowski-opensource

Copy link
Copy Markdown
Member

This PR was merged into the repository by commit 785c3c1.

pkozlowski-opensource pushed a commit that referenced this pull request Apr 16, 2024
…hange detections (#55290)

This commit ensures that manually calling ApplicationRef.tick will
result in any scheduled change detections being canceled. There is no
need for the scheduled one to run because it was manually done by the
`tick` already.

PR Close #55290
@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 May 17, 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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note 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