Skip to content

fix(bg): success redirect only after reconnect post-process done#1358

Closed
sidvishnoi wants to merge 2 commits intomainfrom
reconnect-redirect-in-end
Closed

fix(bg): success redirect only after reconnect post-process done#1358
sidvishnoi wants to merge 2 commits intomainfrom
reconnect-redirect-in-end

Conversation

@sidvishnoi
Copy link
Copy Markdown
Member

Context

Follow-up for #1357 (UX improvement)

Changes proposed in this pull request

Redirect user to success only after reconnect post-processing is finished.

@sidvishnoi sidvishnoi requested a review from DarianM April 15, 2026 13:52
@github-actions github-actions Bot added the area: background Improvements or additions to extension background script label Apr 15, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 15, 2026

Extension builds preview

Name Link
Latest commit 6ae3111
Latest job logs Run #24475474462
Chrome (1.38MB)Download
Firefox (1.38MB)Download
Safari (2.32MB)Download

@sidvishnoi
Copy link
Copy Markdown
Member Author

Thinking about this more in context of #1329. I think the delay can be substantial for user to be waiting for success-redirect. Let's not do this.

@sidvishnoi sidvishnoi closed this Apr 16, 2026
@sidvishnoi sidvishnoi deleted the reconnect-redirect-in-end branch April 16, 2026 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: background Improvements or additions to extension background script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants