-
Notifications
You must be signed in to change notification settings - Fork 1
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
Read #5
Comments
@soywod Have you started to work on these two issues yet? If not, should I pick one? And can you create a PR-Draft for this issue please for the kanban board? |
I did not start yet, you can take both if you have time! Because I will work on the refactor as mentioned there #9 (comment). |
Alright, then I'll take over those two issues! |
@soywod I've got a question for the first task: Isn't it possible to create a key |
Initially, I did not put the attachments for all messages because of perfs. To get informations about attachments you need to parse bodies, which implies a bigger IMAP request and longer parsing time. What could fix this is a caching system, but as I already explained this will move to a dedicated lib (see pimalaya#118 (reply in thread)). |
Hold on. My fork here get's everything of a message, that also means all attachments of a message. Is this fine? |
Well, we will see by usage. If it's too slow to show the message list, we will remove it! |
Closed due to this PR. |
hasAttachment
)text/html
version of the mail if notext/plain
is found.The text was updated successfully, but these errors were encountered: