Skip to content

collab_panel: Fix favorite channels not surviving startup#52541

Merged
danilo-leal merged 1 commit intomainfrom
collab-favorites-fix
Mar 26, 2026
Merged

collab_panel: Fix favorite channels not surviving startup#52541
danilo-leal merged 1 commit intomainfrom
collab-favorites-fix

Conversation

@danilo-leal
Copy link
Copy Markdown
Member

Follow up to #52378

This PR fixes a little race condition that was happening where we were running the favorite channel pruning function faster than the channels could load, leading to favorite channels not surviving the app restarting. The fix is to make the pruning happen only when the number of channels is bigger than 0, which means the list from the server has already been loaded.
Release Notes:

  • N/A (No release notes yet because this feature hasn't reached the wider public)

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Mar 26, 2026
@zed-codeowner-coordinator zed-codeowner-coordinator bot requested review from a team, kubkon and reflectronic and removed request for a team March 26, 2026 22:16
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Mar 26, 2026
@danilo-leal danilo-leal requested a review from cole-miller March 26, 2026 22:21
@danilo-leal danilo-leal merged commit 1625f98 into main Mar 26, 2026
42 checks passed
@danilo-leal danilo-leal deleted the collab-favorites-fix branch March 26, 2026 22:26
danilo-leal added a commit that referenced this pull request Mar 30, 2026
Follow up to #52541

Favorite channels weren't appearing on multiple windows because they
were serialized in a given window's collab panel state. This PR moves
them to the global key value store. Also, solved a little issue with
non-unique ids which was making clicking on the original instance of a
favorite channel do nothing.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants