runbox7 icon indicating copy to clipboard operation
runbox7 copied to clipboard

Messages can not be deleted from the list with INBOX vs Inbox mismatch (without local index)

Open gtandersen opened this issue 3 years ago • 10 comments
trafficstars

Describe the bug After importing a number of messages via IMAP and attempting to delete a message using the message action menu buttons, no response is visible in the app.

To Reproduce Steps to reproduce the behavior:

  1. Import a number of email via IMAP
  2. Go to https://runbox.com/app
  3. Select a message in the list and click to delete it
  4. Observe that no response is given in the app
  5. Observe the following error in the console: TypeError: i.folderCounts[r.folder] is undefined

Expected behavior The selected message should be deleted when clicking the delete/trash button.

gtandersen avatar Feb 02 '22 18:02 gtandersen

When trying to delete the same messages that have been imported via IMAP using a checkbox and the floating mail list menu delete button the message disappears and then re-appears a few seconds later. So the same result is the outcome but it looks different to the user as at least something happens.

runboxdave avatar Feb 02 '22 18:02 runboxdave

After further investigation it seems that this problem is related to the messages being assigned to a folder called INBOX rather than Inbox. The messages that were in INBOX were imported using Apple Mail (whether this applies to other email clients or the Runbox IMAP Import is not known yet).

runboxdave avatar Feb 07 '22 10:02 runboxdave

Have confirmed that importing messages via Claws Mail creates the same issue (INBOX folder not Inbox, and can't delete)

castaway avatar Feb 07 '22 11:02 castaway

This is related to #69 and internal issue 978.

gtandersen avatar Feb 07 '22 17:02 gtandersen

I'm seeing this same behavior (as the OP) as well. @gtandersen you say it's related to #69 but that issue was closed and from what I read in the issue, nothing was done. If #69 is still an active issue, can it be re-opened?

rootwork avatar Feb 08 '22 02:02 rootwork

#69 was resolved, that ensured we could display messages that're in folder INBOX (as well as Inbox), if you do a search you'll see results from both. It didn't fix the underlying issue though.

castaway avatar Feb 09 '22 10:02 castaway

An idea for fixing this one (or at least papering over it until we get backend import issue sorted): The "verifymessages" system, which is used when api folder counts dont match index folder counts (as they wont with INBOX/Inbox being diffferent) - is used to flag messages with different settings (eg read/unread, flagged etc) for reindexing. We could also have it sort out the folder name too (and flag that for reindexing as well).

castaway avatar Feb 23 '22 10:02 castaway

This should hopefully improve with #1215 (right @castaway?).

gtandersen avatar Mar 02 '22 13:03 gtandersen

Nope, that one only turns verification back on, would need another for the INBOX/Inbox idea

castaway avatar Mar 03 '22 12:03 castaway

MR submitted to backend to tackle this issue as described above

castaway avatar Mar 10 '22 10:03 castaway