Skip to content

refactor(core): Update Transition finished promise to not throw unhan…#60816

Closed
atscott wants to merge 1 commit intoangular:mainfrom
atscott:finishedreject
Closed

refactor(core): Update Transition finished promise to not throw unhan…#60816
atscott wants to merge 1 commit intoangular:mainfrom
atscott:finishedreject

Conversation

@atscott
Copy link
Copy Markdown
Contributor

@atscott atscott commented Apr 9, 2025

…dled rejection

As with InternalNavigationResult, this commit adds the catch to the finished promise on InternalNavigationTransition to avoid unhandled promise rejections.

…dled rejection

As with `InternalNavigationResult`, this commit adds the `catch` to the
`finished` promise on `InternalNavigationTransition` to avoid unhandled
promise rejections.
@atscott atscott requested a review from tbondwilkinson April 9, 2025 20:48
@pullapprove pullapprove bot added the requires: TGP This PR requires a passing TGP before merging is allowed label Apr 9, 2025
@atscott atscott added target: patch This PR is targeted for the next patch release and removed requires: TGP This PR requires a passing TGP before merging is allowed labels Apr 9, 2025
@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Apr 9, 2025
@pullapprove pullapprove bot added requires: TGP This PR requires a passing TGP before merging is allowed and removed area: core Issues related to the framework runtime labels Apr 9, 2025
@ngbot ngbot bot modified the milestone: Backlog Apr 9, 2025
@kirjs kirjs added the area: core Issues related to the framework runtime label Apr 10, 2025
@ngbot ngbot bot added this to the Backlog milestone Apr 10, 2025
@atscott
Copy link
Copy Markdown
Contributor Author

atscott commented Apr 13, 2025

@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Apr 13, 2025
@kirjs
Copy link
Copy Markdown
Contributor

kirjs commented Apr 15, 2025

@kirjs
Copy link
Copy Markdown
Contributor

kirjs commented Apr 15, 2025

This PR was merged into the repository by commit e90daf3.

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

@kirjs kirjs closed this in e90daf3 Apr 15, 2025
kirjs pushed a commit that referenced this pull request Apr 15, 2025
…dled rejection (#60816)

As with `InternalNavigationResult`, this commit adds the `catch` to the
`finished` promise on `InternalNavigationTransition` to avoid unhandled
promise rejections.

PR Close #60816
@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 16, 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 requires: TGP This PR requires a passing TGP before merging is allowed 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