TF-2536 Add adr for logic to get all mailbox
Signed-off-by: dab246 <tdvu@linagora.com>
This commit is contained in:
@@ -0,0 +1,25 @@
|
||||
# 34. Change the logic to get all mailbox to avoid miss mailbox (#2536)
|
||||
|
||||
Date: 2024-02-05
|
||||
|
||||
## Status
|
||||
|
||||
Accepted
|
||||
|
||||
## Context
|
||||
|
||||
- Currently `MailboxRepository` get the data from local first:
|
||||
- if cache hit `(have mailbox + cache)`, `/changes` to update
|
||||
- if cache miss `(no mailbox, no state)`, `/get` to update view
|
||||
|
||||
What happen if cache hit, but not have all the data? Yep, this is the issue.
|
||||
|
||||
## Decision
|
||||
|
||||
- Change the logic to get all mailbox
|
||||
- Get from the local `(support quick response and offline mode)` -> yield to UI
|
||||
- Get `Mailbox/get` from JMAP to update the list of mailbox correctly -> yield to UI
|
||||
|
||||
## Consequences
|
||||
|
||||
- No more lost mailboxes. The latest mailbox list is always updated.
|
||||
Reference in New Issue
Block a user