recent_projects: Fix remote reconnect when server is not running#49834
Conversation
When an existing remote workspace's connection is dead (e.g. server died and reconnect failed, leaving the client in ServerNotRunning state), `remote_connection()` returns `None`. Previously this caused an error, blocking reconnection. Now the code falls through to establish a fresh connection instead.
|
We require contributors to sign our Contributor License Agreement, and we don't have @mangas on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'. |
|
@cla-bot check |
|
We require contributors to sign our Contributor License Agreement, and we don't have @mangas on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'. |
|
The cla-bot has been summoned, and re-checked this pull request! |
|
Tested manually and it works for me. Let me know if there's anything else to get it merged. |
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
|
|
/cherry-pick preview |
) Closes #49363 When an existing remote workspace's connection is dead (e.g. the server died and reconnect failed, leaving the client in `ServerNotRunning` state), `remote_connection()` returns `None`. Previously this caused an error dialog, blocking reconnection — the user had to manually switch to another project and back to recover. Now the code falls through to establish a fresh connection instead, matching the previous behavior where clicking "Reconnect" would just work. Release Notes: - Fixed remote reconnect failing with an error when the server is not running, now establishes a fresh connection instead.
|
🍒💥 Cherry-pick did not succeed |
) Closes #49363 When an existing remote workspace's connection is dead (e.g. the server died and reconnect failed, leaving the client in `ServerNotRunning` state), `remote_connection()` returns `None`. Previously this caused an error dialog, blocking reconnection — the user had to manually switch to another project and back to recover. Now the code falls through to establish a fresh connection instead, matching the previous behavior where clicking "Reconnect" would just work. Release Notes: - Fixed remote reconnect failing with an error when the server is not running, now establishes a fresh connection instead.
) (cherry-pick to preview) (#49871) Cherry-pick of #49834 to preview ---- Closes #49363 When an existing remote workspace's connection is dead (e.g. the server died and reconnect failed, leaving the client in `ServerNotRunning` state), `remote_connection()` returns `None`. Previously this caused an error dialog, blocking reconnection — the user had to manually switch to another project and back to recover. Now the code falls through to establish a fresh connection instead, matching the previous behavior where clicking "Reconnect" would just work. Release Notes: - Fixed remote reconnect failing with an error when the server is not running, now establishes a fresh connection instead. Co-authored-by: Filipe Azevedo <filipe@azevedo.io>
…-industries#49834) Closes zed-industries#49363 When an existing remote workspace's connection is dead (e.g. the server died and reconnect failed, leaving the client in `ServerNotRunning` state), `remote_connection()` returns `None`. Previously this caused an error dialog, blocking reconnection — the user had to manually switch to another project and back to recover. Now the code falls through to establish a fresh connection instead, matching the previous behavior where clicking "Reconnect" would just work. Release Notes: - Fixed remote reconnect failing with an error when the server is not running, now establishes a fresh connection instead.
Closes #49363
When an existing remote workspace's connection is dead (e.g. the server died and reconnect failed, leaving the client in
ServerNotRunningstate),remote_connection()returnsNone. Previously this caused an error dialog, blocking reconnection — the user had to manually switch to another project and back to recover.Now the code falls through to establish a fresh connection instead, matching the previous behavior where clicking "Reconnect" would just work.
Release Notes: