-
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
Add withErrorHandler
feature to provideRouter
#48551
Conversation
52f01f9
to
929be8d
Compare
929be8d
to
f89e13c
Compare
`withNavigationErrorHandler` is a close replacement for the `RouterModule.forRoot.errorHandler` / `Router.errorHandler`. It provides a quick, short way for users to define a function to handle `NavigationError` events.
f89e13c
to
981807d
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 with one comment about an alias 👍
`errorHandler` and `malformedUriErrorHandler` were already deprecated in the Router class public API. This change updates the deprecation to extend to `RouterModule.forRoot` configuration options.
981807d
to
e89378e
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.
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
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
merge assistance: Nothing is failing but checks aren't completing |
This PR was merged into the repository by commit 15eccef. |
`errorHandler` and `malformedUriErrorHandler` were already deprecated in the Router class public API. This change updates the deprecation to extend to `RouterModule.forRoot` configuration options. PR Close #48551
…er (angular#48551) `withNavigationErrorHandler` is a close replacement for the `RouterModule.forRoot.errorHandler` / `Router.errorHandler`. It provides a quick, short way for users to define a function to handle `NavigationError` events. PR Close angular#48551
`errorHandler` and `malformedUriErrorHandler` were already deprecated in the Router class public API. This change updates the deprecation to extend to `RouterModule.forRoot` configuration options. PR Close angular#48551
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. |
Commit 1:
withErrorHandler
is a close replacement for theRouterModule.forRoot.errorHandler
/Router.errorHandler
.It provides a quick, short way for users to define a function to handle
NavigationError
events.Commit 2:
errorHandler
andmalformedUriErrorHandler
were already deprecated inthe Router class public API. This change updates the deprecation to
extend to
RouterModule.forRoot
configuration options.