Skip to content

refactor(core): Reorder resolve/reject in relation to navigate[error/…#63377

Closed
atscott wants to merge 1 commit into
angular:mainfrom
atscott:eventordernavigationapi
Closed

refactor(core): Reorder resolve/reject in relation to navigate[error/…#63377
atscott wants to merge 1 commit into
angular:mainfrom
atscott:eventordernavigationapi

Conversation

@atscott

@atscott atscott commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

…success]

Order was changed in whatwg/html#11512 (see that PR for details as to why).

Note that the transition's resolve/reject still happens after the event, per the spec.

https://whatpr.org/html/10919/nav-history-apis.html#abort-a-navigateevent

@atscott atscott added the target: patch This PR is targeted for the next patch release label Aug 25, 2025
@pullapprove pullapprove Bot requested a review from csmick August 25, 2025 18:02
@pullapprove pullapprove Bot added requires: TGP This PR requires a passing TGP before merging is allowed target: patch This PR is targeted for the next patch release and removed target: patch This PR is targeted for the next patch release labels Aug 25, 2025
@angular-robot angular-robot Bot added the area: core Issues related to the framework runtime label Aug 25, 2025
@ngbot ngbot Bot added this to the Backlog milestone Aug 25, 2025
@atscott atscott removed the requires: TGP This PR requires a passing TGP before merging is allowed label Aug 26, 2025
@pullapprove pullapprove Bot added the requires: TGP This PR requires a passing TGP before merging is allowed label Aug 26, 2025
@atscott atscott added action: merge The PR is ready for merge by the caretaker and removed requires: TGP This PR requires a passing TGP before merging is allowed labels Aug 26, 2025
@pullapprove pullapprove Bot added the requires: TGP This PR requires a passing TGP before merging is allowed label Aug 26, 2025
@atscott atscott requested a review from tbondwilkinson August 26, 2025 21:14
@atscott

atscott commented Aug 27, 2025

Copy link
Copy Markdown
Contributor Author

@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 Aug 27, 2025
@atscott

atscott commented Aug 27, 2025

Copy link
Copy Markdown
Contributor Author

caretaker note: Current approvals are sufficient

@AndrewKushnir

Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

@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 Sep 27, 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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note 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.

2 participants