Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Clean button, "Empty Trash/Spam now" banner and Filters when there is no email in Spam/Trash folder #2951

Closed
dieptran88 opened this issue Jun 20, 2024 · 6 comments
Assignees
Labels

Comments

@dieptran88
Copy link
Collaborator

Desc:

  • When there is no email in Trash, Spam folder, remove:
    • Clean button at the folder menu
    • Empty Trash/Spam now when open the folder
    • Remove/Disable Filters, Select all message, Recover deleted messages button when open the folder:

image

@hoangdat
Copy link
Member

@hoangdat
Copy link
Member

hi @chibenwa , @Arsnael
we still have the problem of inconsistency data for Mailbox/get Email/query and Email/get

  1. For the case of Trash: Mailbox/get method response: totalEmails = 4

Image

  1. Email/query also return 4 ids, but Email/get responses 4 ids in notFound

Image

@Arsnael
Copy link
Member

Arsnael commented Jul 17, 2024

Environment? Platform? User?

Maybe an inconsistency between the email query view and the email table but it's hard to go to check if you don't give us a minimum of details to where we can look :)

@hoangdat
Copy link
Member

  • Environment: Prod of linagora - it happens in @dieptran88 and some QA guys

@hoangdat
Copy link
Member

hoangdat commented Aug 5, 2024

hi @Arsnael please help us to take a look on it #2951 (comment)
Env: tmail.linagora.com

@Arsnael
Copy link
Member

Arsnael commented Aug 5, 2024

Probably a task to run to fix inconsistencies... probably a mix of quota recomputation and emailqueryview

This is prod as well, and running such tasks are heavy on the system. I am not keen on taking such responsibility for a wrong mail counter sorry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants