Skip to content

refactor(core): update FakeNavigation to the latest spec#62017

Closed
atscott wants to merge 2 commits into
angular:mainfrom
atscott:fakenavigationupdatesjune11
Closed

refactor(core): update FakeNavigation to the latest spec#62017
atscott wants to merge 2 commits into
angular:mainfrom
atscott:fakenavigationupdatesjune11

Conversation

@atscott

@atscott atscott commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

Spec updates are in whatwg/html#10919

For the most part, the updates revolve around the deferred commit handling (with precommitHandler). Updates to redirect allow more options. A committed promise now exists on the transition since commits can be delayed. Tests were made zoneless for easier debugging and timeouts were reduced.

@atscott atscott added target: patch This PR is targeted for the next patch release requires: TGP This PR requires a passing TGP before merging is allowed labels Jun 11, 2025
@pullapprove pullapprove Bot requested a review from tbondwilkinson June 11, 2025 20:50
@pullapprove pullapprove Bot removed the target: patch This PR is targeted for the next patch release label Jun 11, 2025
@angular-robot angular-robot Bot added the area: core Issues related to the framework runtime label Jun 11, 2025
@ngbot ngbot Bot added this to the Backlog milestone Jun 11, 2025
@atscott atscott force-pushed the fakenavigationupdatesjune11 branch from 45e41dd to 63e7890 Compare June 11, 2025 20:50
Comment thread packages/core/primitives/dom-navigation/testing/fake_navigation.ts Outdated
Comment thread packages/core/primitives/dom-navigation/testing/fake_navigation.ts Outdated
Comment thread packages/core/primitives/dom-navigation/testing/fake_navigation.ts
Comment thread packages/core/primitives/dom-navigation/testing/fake_navigation.ts Outdated
@thePunderWoman

thePunderWoman commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

TESTED=TGP

@atscott atscott force-pushed the fakenavigationupdatesjune11 branch from 63e7890 to dd97d49 Compare June 16, 2025 18:27
@atscott atscott added action: merge The PR is ready for merge by the caretaker 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 Jun 17, 2025
@pullapprove pullapprove Bot added the requires: TGP This PR requires a passing TGP before merging is allowed label Jun 17, 2025
@pkozlowski-opensource pkozlowski-opensource added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: merge The PR is ready for merge by the caretaker labels Jun 18, 2025
@pkozlowski-opensource

Copy link
Copy Markdown
Member

Looks like it needs rebase so moving it from merge => cleanup

Spec updates are in whatwg/html#10919

For the most part, the updates revolve around the deferred commit
handling (with precommitHandler). Updates to redirect allow more
options. A committed promise now exists on the transition since commits
can be delayed. Tests were made zoneless for easier debugging and
timeouts were reduced.
@atscott atscott force-pushed the fakenavigationupdatesjune11 branch from acca5ca to fc14fc2 Compare June 18, 2025 16:23
@atscott atscott added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Jun 18, 2025
@atscott atscott force-pushed the fakenavigationupdatesjune11 branch from fc14fc2 to 1a49d06 Compare June 18, 2025 23:16
@crisbeto

Copy link
Copy Markdown
Member

I'll proceed with merging this. The adev test failure looks unrelated.

@crisbeto

Copy link
Copy Markdown
Member

This PR was merged into the repository by commit fa5ae92.

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

@crisbeto crisbeto closed this in fa5ae92 Jun 19, 2025
crisbeto pushed a commit that referenced this pull request Jun 19, 2025
Spec updates are in whatwg/html#10919

For the most part, the updates revolve around the deferred commit
handling (with precommitHandler). Updates to redirect allow more
options. A committed promise now exists on the transition since commits
can be delayed. Tests were made zoneless for easier debugging and
timeouts were reduced.

PR Close #62017
@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 Jul 20, 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.

5 participants