-
Notifications
You must be signed in to change notification settings - Fork 260
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
Issues with drafts [$65 awarded] #9
Comments
I use the Mail app with my Gmail account. I notice a few additional issues:
|
I created an IMAP account with my hoster, and in their webmail (Roundcube) if I save a draft without a recipient, then Nextcloud Mail will say: "Something's wrong with loading your drafts :/" (manual translation). After adding a recipient in my hoster's Roundcube, Nextcloud Mail does load my drafts properly. < nextcloud> < system> < version>11.0.2.7 |
Is this solved? I have close to 200 draft emails in my mail app on nextcloud instance. I've replied to less than 10 mails that I can recall while using the mail app. I have no idea why so many drafts would be saved. I usually just read the mail and delete the ones I don't need. |
@ChristophWurst @irgendwie @Gomez can anyone of you look into this? :) It’s one of the most annoying bugs I encounter daily. And it actually punishes the people who use the mail app a lot. :\ |
Actually, no drafts are even saved on my environment using the master branch (it's working with the last stable version). The console gives me the message:
It seems " Does anyone reproduce this? It seems that these changes in the composerview are a bit old... Or maybe nobody uses the drafts 😄
😝 |
@lcalaresu if you can look into this it would be much appreciated. ;) I think @ChristopWurst said he doesn't use drafts so much. |
No, I don't write any mails at all, you know 😛 |
@jancborchardt |
Sure, happy to help design & interaction-wise as far as I can, and I guess @ChristophWurst / @Gomez / @nextcloud/mail can answer technical questions. :) |
Just added another 50$ to the bounty! :) Any takers? |
@jancborchardt could you please try the latest master (which includes #420) and check which is items have been fixed already? Thanks. |
Will do as soon as I have the Nextcloud dev environment set up on my new laptop! :) |
Those two are wontfixes for 0.7. I've ticket the ones that I'm unable to reproduce, probably because #420 fixed them. |
@ChristophWurst yup, totally fine. The only important one left is the
cause that breaks drafts quite hard. |
Moved the two other issues which are for after 0.7 to:
|
As discussed, I've spent quite some time on debugging this and I've not yet found a way to fix the encoding. |
Apparently the horde libs do not produce a valid MIME unless you actually use the 'send' method. Since they fortunatelly have a null transport with an empty impl, we can use this to have the same encoding magic going on as we do have in place for actually sending messages via SMTP. Now drafts stored to IMAP are encoded correctly. Fixes #9 Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and questions. |
There are a bunch of things problematic with drafts, and we should fix them. Cause drafts are a pretty helpful feature to save your stuff and sync it.
Sorted by severity:
>
is escaped which is horrible for quoted textFor later:
The $65 bounty on this issue has been claimed at Bountysource.
The text was updated successfully, but these errors were encountered: