-
Notifications
You must be signed in to change notification settings - Fork 322
Closed
Labels
a-apiImplementing specific parts of the Zulip server APIImplementing specific parts of the Zulip server APIa-modelImplementing our data model (PerAccountStore, etc.)Implementing our data model (PerAccountStore, etc.)help wanted
Milestone
Description
With #787 we implemented most of #150, handling moved messages. But as that PR description said:
Fixes most of #150. (After this lands we should do a quick scan to identify anything else that needs updating on message moves — there are several new data structures in the store that we've merged in recent weeks.)
Then we didn't do that scan.
Places that still need updating on message moves include:
-
Unreads
(which has a TODO comment for it) -
→ split to its own issue: Handle moved messages in recent-senders data #1460RecentSenders
(that's one of those recently-merged data structures) - … OK and I think that's it, having now scanned through what's in the store. (Another recently-merged data structure is
RecentDmConversationsView
, but that's only about DMs and those can't be moved.)
Related issues:
Metadata
Metadata
Assignees
Labels
a-apiImplementing specific parts of the Zulip server APIImplementing specific parts of the Zulip server APIa-modelImplementing our data model (PerAccountStore, etc.)Implementing our data model (PerAccountStore, etc.)help wanted
Type
Projects
Status
Done