Tryton - Issues

 

Issue9456

Title Add option to raise error if sendmail fails
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type behavior Components trytond
Assigned To ced Keywords review
Reviews 298071002
View: 298071002

Created on 2020-07-03.16:56:32 by ced, last changed by roundup-bot.

Messages
New changeset b32a7c5d6a00 by Cédric Krier in branch 'default':
Add option to ensure emails are sent
https://hg.tryton.org/tryton-env/rev/b32a7c5d6a00
New changeset 8d3d023fc121 by Cédric Krier in branch 'default':
Add option to ensure emails are sent
https://hg.tryton.org/trytond/rev/8d3d023fc121
review298071002 updated at https://codereview.tryton.org/298071002/#ps317851002
review298071002 updated at https://codereview.tryton.org/298071002/#ps301891002
review298071002 updated at https://codereview.tryton.org/298071002/#ps306001002
msg58948 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2020-07-03.16:56:31
sendmail does not raise exception in order to not stop the transaction because of smtp issue.
But sometimes we want to be sure that an email will be sent or the transaction must be roll-backed.
I propose to add a strict parameter which will make sendmail to raise exceptions.
History
Date User Action Args
2020-07-21 14:34:21roundup-botsetmessages: + msg59410
2020-07-21 14:34:15roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg59409
2020-07-05 18:11:16reviewbotsetmessages: + msg59002
2020-07-05 14:15:52reviewbotsetmessages: + msg59001
2020-07-03 17:13:31reviewbotsetnosy: + reviewbot
messages: + msg58950
2020-07-03 17:04:50cedlinkissue9449 superseder
2020-07-03 17:01:24cedsetstatus: in-progress -> testing
reviews: 298071002
keyword: + review
2020-07-03 16:56:32cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)