Skip to content
This repository was archived by the owner on Aug 21, 2026. It is now read-only.

Add an explainer for navigateEvent.redirect() - #279

Merged
domenic merged 1 commit into
mainfrom
redirect
Feb 10, 2025
Merged

domenic merged 1 commit into
mainfrom
redirect

Conversation

@domenic

@domenic domenic commented Feb 7, 2025 •

Copy link
Copy Markdown
Collaborator

Part of #124. This replaces the previous idea of navigation.transition.redirect(), with something that is only scoped to the { commit: "after-transition" } case for now. A future extension to { commit: "immediate" } is gestured at.

Also remove the explainer section for rollback, since we've had fewer requests for it. We can keep #124 open to discuss it though.

Part of #124. This replaces the previous idea of navigation.transition.rollback(), with something that is only scoped to the { commit: "after-transition" } case for now. A future extension to { commit: "immediate" } is gestured at.

Also remove the explainer section for rollback, since we've had fewer requests for it. We can keep #124 open to discuss it though.
@domenic
domenic merged commit 51bbee2 into main Feb 10, 2025
@domenic
domenic deleted the redirect branch February 10, 2025 05:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants