Skip to content

Commit

Permalink
msgstore: fix new-message bell for threaded directories
Browse files Browse the repository at this point in the history
Threaded directories were never marked as loaded and thus the terminal
bell was never rung when a new message arrived in them. Add the missing
directive to mark them as loaded when appropriate.

Fixes: 8d47047 ("msgstore: do not trigger notification on initial load")
Changelog-fixed: The new-message bell is now rung again for threaded
 directories as well.
Cc: Remko Tronçon <[email protected]>
Signed-off-by: Karel Balej <[email protected]>
Acked-by: Robin Jarry <[email protected]>
  • Loading branch information
balejk authored and rjarry committed Jan 23, 2025
1 parent 4c7a22c commit 1d5551b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lib/msgstore.go
Original file line number Diff line number Diff line change
Expand Up @@ -309,6 +309,7 @@ func (store *MessageStore) Update(msg types.WorkerMessage) {

store.Messages = newMap
update = true
store.directoryContentsLoaded = true
case *types.MessageInfo:
infoUpdated := msg.Info.Envelope != nil || msg.Info.Error != nil
if existing, ok := store.Messages[msg.Info.Uid]; ok && existing != nil {
Expand Down

0 comments on commit 1d5551b

Please sign in to comment.