Skip to content

refactor(router): Warn if a navigation will change in the upcoming v16 release#48688

Closed
atscott wants to merge 2 commits intoangular:mainfrom
atscott:warnOnDifferentUrl
Closed

refactor(router): Warn if a navigation will change in the upcoming v16 release#48688
atscott wants to merge 2 commits intoangular:mainfrom
atscott:warnOnDifferentUrl

Conversation

@atscott
Copy link
Contributor

@atscott atscott commented Jan 10, 2023

v16 will have a breaking change to the way UrlTrees are constructed. This change is actually a bug fix that makes UrlTree creation correct in more scenarios (see #48508). However, this can affect applications that are relying on the current incorrect behavior. This commit adds a dev mode warning when the target of a navigation will change once #48508 is submitted.

@atscott atscott added area: router target: patch This PR is targeted for the next patch release labels Jan 10, 2023
@ngbot ngbot bot added this to the Backlog milestone Jan 10, 2023
…6 release

v16 will have a breaking change to the way `UrlTree`s are constructed.
This change is actually a bug fix that makes `UrlTree` creation correct
in more scenarios (see angular#48508). However, this can affect applications that are
relying on the current incorrect behavior. This commit adds a dev mode
warning when the target of a navigation will change once angular#48508 is
submitted.
@atscott
Copy link
Contributor Author

atscott commented Jan 12, 2023

Blocked until root cause is found for b/265215141

@atscott atscott added target: minor This PR is targeted for the next minor release action: merge The PR is ready for merge by the caretaker and removed state: blocked target: patch This PR is targeted for the next patch release labels Feb 7, 2023
Copy link
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@atscott
Copy link
Contributor Author

atscott commented Feb 7, 2023

This PR was merged into the repository by commit 5128ba0.

@angular-automatic-lock-bot
Copy link

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 Mar 10, 2023
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: router target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants