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
feat(router): add a migration to remove relativeLinkResolution
usages
#47604
feat(router): add a migration to remove relativeLinkResolution
usages
#47604
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Could you add a g3 version as well? While we can just do a find/replace, it might still be easier to run the migration.
0498f5d
to
5cf2142
Compare
@atscott I've added a google3 migration logic as requested. Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Note: this PR needs some additional work to separate lookups from writes, based on #47599 (comment), adding the "blocked" label for now. |
5cf2142
to
b936eec
Compare
This PR is now updated based on the feedback from @crisbeto. The PR is ready for re-review. Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
b936eec
to
10a7f1b
Compare
As of Angular v15, the deprecated `relativeLinkResolution` config option of the Router is removed. This migration cleans up (removes) the `relativeLinkResolution` fields from the Router config objects in applications code. ```ts import { RouterModule } from '@angular/router'; RouterModule.forRoot([], { relativeLinkResolution: 'legacy', enableTracing: false, }); ``` ```ts import { RouterModule } from '@angular/router'; RouterModule.forRoot([], { // the `relativeLinkResolution` is removed enableTracing: false, }); ```
10a7f1b
to
8d21eac
Compare
Merge-assistance: this PR would require an extra patch (cl/479482886) applied to the sync CL. |
This PR was merged into the repository by commit 7bee28d. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
As of Angular v15, the deprecated
relativeLinkResolution
config option of the Router is removed. This migration cleans up (removes) therelativeLinkResolution
fields from the Router config objects in applications code.PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?