Skip to content

fix(router): Remove deprecated Router properties #51502

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

Closed
wants to merge 1 commit into from

Conversation

atscott
Copy link
Contributor

@atscott atscott commented Aug 25, 2023

This commit removes deprecated properties on the Router. These are meant to be configured through DI and not meant to be changed during runtime.

BREAKING CHANGE: The following Router properties have been removed from the public API:

  • canceledNavigationResolution
  • paramsInheritanceStrategy
  • titleStrategy
  • urlUpdateStrategy
  • malformedUriErrorHandler

These should instead be configured through the provideRouter or RouterModule.forRoot APIs.

@atscott atscott added the target: major This PR is targeted for the next major release label Aug 25, 2023
@atscott atscott requested a review from AndrewKushnir August 25, 2023 15:43
@angular-robot angular-robot bot added the detected: breaking change PR contains a commit with a breaking change label Aug 25, 2023
@JeanMeche
Copy link
Member

Nit: You could also update the deprecation guide in this PR also.

This commit removes deprecated properties on the Router. These are meant
to be configured through DI and not meant to be changed during runtime.

BREAKING CHANGE: The following Router properties have been removed from
the public API:

- canceledNavigationResolution
- paramsInheritanceStrategy
- titleStrategy
- urlUpdateStrategy
- malformedUriErrorHandler

These should instead be configured through the `provideRouter` or
`RouterModule.forRoot` APIs.
@atscott atscott force-pushed the removedeprecations branch from 5cfbfb0 to 601fc14 Compare August 25, 2023 15:55
@ngbot ngbot bot added this to the Backlog milestone Aug 25, 2023
Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a 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

@pullapprove pullapprove bot requested a review from alxhub August 25, 2023 18:55
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.

Reviewed-for: public-api

@atscott atscott removed request for alxhub and dylhunn August 29, 2023 17:00
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Aug 29, 2023
@jessicajaniuk
Copy link
Contributor

This PR was merged into the repository by commit c62e680.

LayZeeDK pushed a commit to LayZeeDK/angular__angular that referenced this pull request Sep 20, 2023
This commit removes deprecated properties on the Router. These are meant
to be configured through DI and not meant to be changed during runtime.

BREAKING CHANGE: The following Router properties have been removed from
the public API:

- canceledNavigationResolution
- paramsInheritanceStrategy
- titleStrategy
- urlUpdateStrategy
- malformedUriErrorHandler

These should instead be configured through the `provideRouter` or
`RouterModule.forRoot` APIs.

PR Close angular#51502
@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 Sep 29, 2023
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
This commit removes deprecated properties on the Router. These are meant
to be configured through DI and not meant to be changed during runtime.

BREAKING CHANGE: The following Router properties have been removed from
the public API:

- canceledNavigationResolution
- paramsInheritanceStrategy
- titleStrategy
- urlUpdateStrategy
- malformedUriErrorHandler

These should instead be configured through the `provideRouter` or
`RouterModule.forRoot` APIs.

PR Close angular#51502
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 detected: breaking change PR contains a commit with a breaking change target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants