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

Imported mails cannot be selected with loadAndSelect in MailViewModel #8305

Open
abdulkadir-tutao opened this issue Jan 17, 2025 · 0 comments
Labels
bug broken functionality, usability problems, unexpected errors

Comments

@abdulkadir-tutao
Copy link
Contributor

Imported mails cannot be selected due to the stop condition relying on mailId comparison in the loadAndSelect function in MailViewModel. Mails are sorted by received date in the MailListModel, but the encoded date in the mailId is the date the mails were created in the server. Therefore, the current stop condition using the mailId is not sufficient to find out whether we have already loaded mails older than the mail we're looking for.

Expected behavior
Implementation of a suitable stop condition that accounts for the imported mails case and relys on the comparison of mailSetEntryElementIds. One approach would be to modify the URL and pass /mailSetElementId/mailSetEntryElementId/ instead of /mailSetElementId/mailElementId/

Version [261.250116.0]

Additional context
MailViewModel.ts

@abdulkadir-tutao abdulkadir-tutao added the bug broken functionality, usability problems, unexpected errors label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug broken functionality, usability problems, unexpected errors
Projects
None yet
Development

No branches or pull requests

1 participant