Skip to content

fix(router): Clear internal transition when navigation finalizes#54261

Closed
atscott wants to merge 2 commits intoangular:mainfrom
atscott:clearNavigationState
Closed

fix(router): Clear internal transition when navigation finalizes#54261
atscott wants to merge 2 commits intoangular:mainfrom
atscott:clearNavigationState

Conversation

@atscott
Copy link
Contributor

@atscott atscott commented Feb 5, 2024

This commit fixes a small memory issue in the router where a destroyed component instance would be retained.

fixes #54241

@atscott atscott added area: router target: patch This PR is targeted for the next patch release action: global presubmit The PR is in need of a google3 global presubmit labels Feb 5, 2024
@ngbot ngbot bot added this to the Backlog milestone Feb 5, 2024
@atscott atscott added action: merge The PR is ready for merge by the caretaker and removed action: global presubmit The PR is in need of a google3 global presubmit labels Feb 6, 2024
@atscott atscott force-pushed the clearNavigationState branch from 3dc7d61 to ae993ca Compare February 6, 2024 19:39
@jessicajaniuk
Copy link
Contributor

This PR was merged into the repository by commit 6681292.

jessicajaniuk pushed a commit that referenced this pull request Feb 6, 2024
)

This commit fixes a small memory issue in the router where a destroyed
component instance would be retained.

fixes #54241

PR Close #54261
jessicajaniuk pushed a commit to jessicajaniuk/angular that referenced this pull request Feb 7, 2024
@jessicajaniuk jessicajaniuk reopened this Feb 7, 2024
jessicajaniuk pushed a commit that referenced this pull request Feb 7, 2024
jessicajaniuk pushed a commit that referenced this pull request Feb 7, 2024
@jessicajaniuk
Copy link
Contributor

Sadly I had to revert this due to a G3 breakage. I'll ping you with the details.

@jessicajaniuk jessicajaniuk added state: blocked and removed action: merge The PR is ready for merge by the caretaker labels Feb 7, 2024
@atscott atscott added action: global presubmit The PR is in need of a google3 global presubmit and removed state: blocked labels Feb 7, 2024
@atscott atscott force-pushed the clearNavigationState branch from c6f4b0a to 2e353c4 Compare February 15, 2024 20:04
This commit fixes a small memory issue in the router where a destroyed
component instance would be retained.

fixes angular#54241
@atscott atscott force-pushed the clearNavigationState branch from 2e353c4 to 96d1db0 Compare February 15, 2024 21:46
@atscott atscott added action: merge The PR is ready for merge by the caretaker and removed action: global presubmit The PR is in need of a google3 global presubmit labels Feb 20, 2024
@alxhub
Copy link
Member

alxhub commented Feb 21, 2024

This PR was merged into the repository by commit d3b273a.

@alxhub alxhub closed this in d3b273a Feb 21, 2024
alxhub pushed a commit that referenced this pull request Feb 21, 2024
)

This commit fixes a small memory issue in the router where a destroyed
component instance would be retained.

fixes #54241

PR Close #54261
@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 Mar 23, 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: router target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Weird route caching, or leak on routed component's properties

4 participants