-
-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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: Wait for restore url navigation to complete before proceeding #11930
Conversation
Co-authored-by: Artur Signell <artur@vaadin.com>
🦋 Changeset detectedLatest commit: 4f435a7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 5 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Co-authored-by: Artur Signell <artur@vaadin.com>
Co-authored-by: Artur Signell <artur@vaadin.com>
🤖 Hello there, We just published version Thanks! |
…11930) Co-authored-by: Artur Signell <artur@vaadin.com>
🤖 Hello there, We just published version Thanks! |
Copies changes from @Artur- from #11620 but repoints to the v6 branch.
Because this relies on behavior of
history
which JSDOM doesn't correctly implement - I'm testing this through a Remix E2E test (which will be pulled into this Repo in RR v7): remix-run/remix#9914