Skip to content

feat(devtools): clean up router tree for stable release #63081

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AleksanderBodurri
Copy link
Member

@AleksanderBodurri AleksanderBodurri commented Aug 11, 2025

Addresses some cleanup items for the router tree:

  • No longer loads router ng global APIs as a side effect of importing the router. Rather this is now a runtime step that occurs when provideRouter is called.
  • No longer depends on router.navigateByUrl in Angular DevTools. There is now a dedicated global util for this
  • Find router instance logic no longer depends on token name
  • Prevents navigating to lazy or redirect routes (these don't have an associated component)

@ngbot ngbot bot added this to the Backlog milestone Aug 11, 2025
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Aug 11, 2025
@AleksanderBodurri AleksanderBodurri force-pushed the router-tree-fixes branch 2 times, most recently from 8cb1f47 to 7af49a9 Compare August 11, 2025 03:48
Addresses some cleanup items for the router tree:

- No longer loads router ng global APIs as a side effect of importing the router. Rather this is now a runtime step that occurs when provideRouter is called.
- No longer depends on router.navigateByUrl in Angular DevTools. There is now a dedicated global util for this
- Router instance logic no longer depends on token name
- Prevents navigating to lazy or redirect routes (these don't have an associated component)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: devtools detected: feature PR contains a feature commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant