[Users] [Bug 3533] DSN (delivery status notification)

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

Olivier Brunel <***@jjacky.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1764|0 |1
is obsolete| |

--- Comment #10 from Olivier Brunel <***@jjacky.com> ---
Created attachment 1909
-->
https://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1909&action=edit
Add Delivery Status Notification (DSN) support

Bumping this with a rebase on 3.17 :)

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

--- Comment #11 from wwp <***@claws-mail.org> ---
Good! Why not defaulting to Failure+Delay in folder properties as it's done in
account prefs and compose window?

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

--- Comment #12 from Olivier Brunel <***@jjacky.com> ---
(In reply to comment #11)

Post by n***@thewildbeast.co.uk
Good! Why not defaulting to Failure+Delay in folder properties as it's done
in account prefs and compose window?

First off, it's only done in account prefs. What you see in the compose window
is what will be applied to that mail, and taken from either the folder
properties if any, else the account prefs -- IOW this isn't a default that's
hard-coded anywhere in the code, it's loaded from user settings.

Now, why not defaulting to Failure+Delay in folder properties as it's done in
account prefs? Well, would that be a question or a suggestion?

If the former, I believe my thinking was that the account pref feels more like
a "general" setting (trying not to say "user default" here to avoid confusion),
where sensible (hard-coded) defaults are good to have, while folder properties
felt more like a specific/"restricted" override of said general setting, thus
it was less important/better as blank.
I also wonder if there couldn't be "confusion" regarding whether those values
were hard-coded defaults or "imported" from an account prefs, and again leaving
it "blank" felt both simpler & easier. :)

If the later, sure, I could do that.

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

--- Comment #13 from wwp <***@claws-mail.org> ---
Thanks for elaborating. Formerly it was a question (to know if it was a bug or
made on purpose, then what purpose?); but it's also a suggestion since I think
it would be more consistent to use the same defaults in every place; and I
don't think we apply strict options in some place and loose ones in other
places for the same feature, elsewhere in CM. Anyway, that is not a big deal at
all.

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

Olivier Brunel <***@jjacky.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1909|0 |1
is obsolete| |

--- Comment #14 from Olivier Brunel <***@jjacky.com> ---
Created attachment 1910
-->
https://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1910&action=edit
Add Delivery Status Notification (DSN) support

v2: Set proper defaults on folder property

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

Olivier Brunel <***@jjacky.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1910|0 |1
is obsolete| |

--- Comment #15 from Olivier Brunel <***@jjacky.com> ---
Created attachment 1912
-->
https://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1912&action=edit
Add Delivery Status Notification (DSN) support

v3: compose: Update current options when changing account

Makes things more consistent with other similar options (e.g. auto-Cc, Privacy
System, ...)

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

wwp <***@claws-mail.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1912|0 |1
is obsolete| |

--- Comment #16 from wwp <***@claws-mail.org> ---
Created attachment 1915
-->
https://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1915&action=edit
Add Delivery Status Notification (DSN) support

Update of the v3 to build against 3.17.0git53

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

wwp <***@claws-mail.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1915|0 |1
is obsolete| |

--- Comment #17 from wwp <***@claws-mail.org> ---
Created attachment 1935
-->
https://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1935&action=edit
Add Delivery Status Notification (DSN) support

Updated to compile against -git176 (2018-11-23).

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

wwp <***@claws-mail.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1935|0 |1
is obsolete| |

--- Comment #18 from wwp <***@claws-mail.org> ---
Created attachment 1936
-->
https://www.thewildbeast.co.uk/claws-mail/bugzilla/attachment.cgi?id=1936&action=edit
Add Delivery Status Notification (DSN) support

Fixes missing diff hunks for compose.h and prefs_folder_item.c, and fix widget
layout packing in prefs_account.

[n***@thewildbeast.co.uk]

https://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3533

Paul <***@claws-mail.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1721|0 |1
is obsolete| |