Skip to content

fix(core): call DestroyRef on destroy callback if view is destroyed [patch] - #61061

Closed
arturovt wants to merge 1 commit into
angular:19.2.xfrom
arturovt:fix/patch_54527
Closed

arturovt wants to merge 1 commit into
angular:19.2.xfrom
arturovt:fix/patch_54527

Conversation

@arturovt

Copy link
Copy Markdown
Contributor

Patch version of 5f7f046

@pullapprove
pullapprove Bot requested a review from alxhub April 30, 2025 14:10
@angular-robot angular-robot Bot added the area: core Issues related to the framework runtime label Apr 30, 2025
@ngbot ngbot Bot added this to the Backlog milestone Apr 30, 2025
@JeanMeche
JeanMeche requested review from thePunderWoman and removed request for alxhub April 30, 2025 14:37

@thePunderWoman thePunderWoman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thePunderWoman thePunderWoman added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Apr 30, 2025
@mmalerba

Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit dbb8702.

The changes were merged into the following branches: 19.2.x

mmalerba pushed a commit that referenced this pull request Apr 30, 2025
@mmalerba mmalerba closed this Apr 30, 2025
@arturovt
arturovt deleted the fix/patch_54527 branch April 30, 2025 15:54
@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 31, 2025
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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants