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): Allow renavigating to failed URLs #43424
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 👍 Just a couple minor comments.
8fd782b
to
7f0721d
Compare
There are situations where the Router does not currently clean up failed navigations correctly. While this is problematic on its own, we can mitigate some of the damage by processing any URL when we get a navigation request when the internal router state is out of sync. Each of the added tests would fail without this change. fixes angular#34795
7f0721d
to
e3f87d8
Compare
There are situations where the Router does not currently clean up failed navigations correctly. While this is problematic on its own, we can mitigate some of the damage by processing any URL when we get a navigation request when the internal router state is out of sync. Each of the added tests would fail without this change. fixes #34795 PR Close #43424
There are situations where the Router does not currently clean up failed navigations correctly. While this is problematic on its own, we can mitigate some of the damage by processing any URL when we get a navigation request when the internal router state is out of sync. Each of the added tests would fail without this change. fixes angular#34795 PR Close angular#43424
There are situations where the Router does not currently clean up failed navigations correctly. While this is problematic on its own, we can mitigate some of the damage by processing any URL when we get a navigation request when the internal router state is out of sync. Each of the added tests would fail without this change. fixes angular#34795 PR Close angular#43424
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. |
There are situations where the Router does not currently clean up failed navigations
correctly. While this is problematic on its own, we can mitigate some of
the damage by processing any URL when we get a navigation request when
the internal router state is out of sync.
Each of the added tests would fail without this change.
fixes #34795
Green TGP