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
fix(router): provide more actionable error message when route is not matched in production mode #53523
Conversation
5dbb106
to
c3fe3ec
Compare
…matched in production mode Prior to this commit when a route is not matched and the application was running in production mode an `[Error]: NG04002` was logged in the console. This however, is not actionable when the application is running on the server where there can be multiple pages being rendered at the same time. Now we change this to also log the route example: `[Error]: NG04002: 'products/Jeep'`. Closes angular#53522
c3fe3ec
to
c5fa9fd
Compare
@@ -58,13 +58,6 @@ export function canLoadFails(route: Route): Observable<LoadedRouterConfig> { | |||
export class ApplyRedirects { | |||
constructor(private urlSerializer: UrlSerializer, private urlTree: UrlTree) {} | |||
|
|||
noMatchError(e: NoMatch): any { |
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.
This was unused.
This PR was merged into the repository by commit 48c5041. |
…matched in production mode (#53523) Prior to this commit when a route is not matched and the application was running in production mode an `[Error]: NG04002` was logged in the console. This however, is not actionable when the application is running on the server where there can be multiple pages being rendered at the same time. Now we change this to also log the route example: `[Error]: NG04002: 'products/Jeep'`. Closes #53522 PR Close #53523
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. |
…matched in production mode (angular#53523) Prior to this commit when a route is not matched and the application was running in production mode an `[Error]: NG04002` was logged in the console. This however, is not actionable when the application is running on the server where there can be multiple pages being rendered at the same time. Now we change this to also log the route example: `[Error]: NG04002: 'products/Jeep'`. Closes angular#53522 PR Close angular#53523
…matched in production mode (angular#53523) Prior to this commit when a route is not matched and the application was running in production mode an `[Error]: NG04002` was logged in the console. This however, is not actionable when the application is running on the server where there can be multiple pages being rendered at the same time. Now we change this to also log the route example: `[Error]: NG04002: 'products/Jeep'`. Closes angular#53522 PR Close angular#53523
…matched in production mode (angular#53523) Prior to this commit when a route is not matched and the application was running in production mode an `[Error]: NG04002` was logged in the console. This however, is not actionable when the application is running on the server where there can be multiple pages being rendered at the same time. Now we change this to also log the route example: `[Error]: NG04002: 'products/Jeep'`. Closes angular#53522 PR Close angular#53523
…matched in production mode (angular#53523) Prior to this commit when a route is not matched and the application was running in production mode an `[Error]: NG04002` was logged in the console. This however, is not actionable when the application is running on the server where there can be multiple pages being rendered at the same time. Now we change this to also log the route example: `[Error]: NG04002: 'products/Jeep'`. Closes angular#53522 PR Close angular#53523
Prior to this commit when a route is not matched and the application was running in production mode an
[Error]: NG04002
was logged in the console. This however, is not actionable when the application is running on the server where there can be multiple pages being rendered at the same time.Now we change this to also log the route example:
[Error]: NG04002: 'products/Jeep'
.Closes #53522