Issue 9320

Title
Add trigger templates
Priority
feature
Status
testing
Nosy list
ced, pokoli, reviewbot, timitos
Assigned to
pokoli
Keywords
review

Created on 2020-05-09.19:44:48 by pokoli, last changed 1 month ago by reviewbot.

Messages

Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2021-08-17.10:45:09

I added more trigger templates on the review. For now I added the states that may be subject to having a notification email to the customer (also some for supplier) but I did not add any other case as I do not want to have a lot of templates (the administrator can create more if required).

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-08-08.17:42:03

I think we need to update the triggers linked to a template if it is changed. Because this is a case that may happen frequently on upgrade (ex: state name change, field is renamed etc.) and the goal of the feature is to make all this internal stuffs transparent to the user.

Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2020-05-09.19:50:34
Here is review299621002 which adds the trigger templates and also defines the templates for the account_invoice module. Once we agreed on the desing I will include the templates for all modules. 

Finally I've decided to hide the configuration fields, I think it will improve the UX because once a template is set the user should not care about any other field. 

Last but not least, I'm a little bit concerned about updating templates. Currently template fields are copied to trigger, so if something changes on template the trigger keeps the old values. I'm wondering if it makes sense to update all trigger values when a template is changed. It won't be to dificult, as we only need to find the related templates, call on_change_template and save the values.
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2020-05-09.19:44:47
History
Date User Action Args
2022-08-11 15:15:18reviewbotsetmessages: + msg77678
2022-03-28 17:30:25reviewbotsetmessages: + msg74833
2021-12-14 16:26:04reviewbotsetmessages: + msg72535
2021-10-27 14:31:06cedlinkissue10913 superseder
2021-08-17 10:55:20reviewbotsetmessages: + msg69402
2021-08-17 10:45:09pokolisetmessages: + msg69401
2021-08-09 09:58:42reviewbotsetmessages: + msg69248
2021-08-08 17:42:03cedsetmessages: + msg69245
nosy: + ced
2020-05-12 13:35:33reviewbotsetmessages: + msg57958
2020-05-09 20:13:32timitossetnosy: + timitos

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