Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@AleksanderBodurri
Copy link
Member

Previously '/' could cause confusion with other routes that use a path here. Since this node is unique in that it is not actually a "Route" that the user configures, we should make it clear with the label that is simply the App Root.

Screenshot 2025-09-21 at 4 53 08 PM

@AleksanderBodurri AleksanderBodurri added target: minor This PR is targeted for the next minor release area: devtools labels Sep 21, 2025
@ngbot ngbot bot added this to the Backlog milestone Sep 21, 2025
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Sep 21, 2025
@AleksanderBodurri AleksanderBodurri force-pushed the router-tree-app-root-as-label branch from 38d9ef8 to 64406aa Compare September 21, 2025 21:03
Previously '/' could cause confusion with other routes that use a path here. I think because this node is unique in that it is not actually a "Route" we should make it clear with the label that is simply the App Root.
@AleksanderBodurri AleksanderBodurri force-pushed the router-tree-app-root-as-label branch from 64406aa to 82761eb Compare October 20, 2025 03:05
@angular-robot angular-robot bot requested a review from JeanMeche October 20, 2025 03:05
@AleksanderBodurri AleksanderBodurri added the action: merge The PR is ready for merge by the caretaker label Oct 25, 2025
@crisbeto crisbeto merged commit ecfcaba into angular:main Oct 27, 2025
19 of 20 checks passed
@crisbeto
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: devtools detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants