-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
refactor(router): Update Router to be providedIn: 'root' #46824
Conversation
235faaf
to
4161b06
Compare
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
Reviewed-for: public-api
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.
Reviewed-for: public-api
This commit updates the Router itself to be `providedIn: 'root'` with a factory function rather than provided in the `RouterModule`.
This PR was merged into the repository by commit d583f85. |
@atscott there was a merge conflict with the 14.1.0 branch, so this PR was merged into the main branch only. Could you please create a new PR and target the 14.1.0 branch if the change should be included there as well? Thank you. |
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. |
This commit updates the Router itself to be
providedIn: 'root'
with afactory function rather than provided in the
RouterModule
.Note: includes commit from #46758 which should be submitted first.