Project

General

Profile

Development #41319

Modèle de mails - Pouvoir mettre des fichiers en PJ

Added by Stéphane Laget about 2 months ago. Updated about 2 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
03 Apr 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Pouvoir, comme l'action de création d'un mail, insérer des pj.
Avec des expressions du type vars().get('reponse_var_pj1_raw'), cela ne devrait pas générer d'erreurs même si le fichier n'existe pas.

0001-mail-templates-add-possibility-to-pre-set-attachment.patch View (9.55 KB) Frédéric Péters, 08 Apr 2020 11:51 AM

Associated revisions

Revision 5917073e (diff)
Added by Frédéric Péters about 2 months ago

mail templates: add possibility to pre-set attachments (#41319)

History

#1 Updated by Frédéric Péters about 2 months ago

Ces fichiers viennent à la place des fichiers qui seraient définis dans l'action, ou en plus ?

(j'étais parti sur l'idée qu'il n'y ait pas de sélection de fichiers quand on est en mode "modèle" puis je me suis dit qu'au contraire ça pouvait avoir du sens d'avoir un modèle et de faire varier le fichier attaché selon le workflow).

#2 Updated by Frédéric Péters about 2 months ago

  • Assignee set to Frédéric Péters

discussion sur le côté : ils s'ajoutent.

#3 Updated by Frédéric Péters about 2 months ago

#4 Updated by Nicolas Roche about 2 months ago

  • Status changed from Solution proposée to Solution validée

Ack.

Juste une remarque : le widget reste caché dans les options avancées quand on ré-édite un modèle alors que des fichiers lui ont déjà été joints. Peut-être modifier ce comportement pour coller à ce qui est fait ailleurs :

 form.add(WidgetList, 'attachments',
     title=_('Attachments (Python expressions)'),
...
-    advanced=True)
+    advanced=self.mail_template.attachments == [])

#5 Updated by Frédéric Péters about 2 months ago

(en fait mon souhait est de revoir ce comportement, parce que les options qui changent d'endroit, je ne trouve plus que ça soit une bonne idée).

#6 Updated by Nicolas Roche about 2 months ago

(ok, alors j'ai rien à ajouter)

#7 Updated by Frédéric Péters about 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 5917073e9ca83328fc701c078914b7be0a680ff6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Apr 8 11:32:03 2020 +0200

    mail templates: add possibility to pre-set attachments (#41319)

#8 Updated by Frédéric Péters about 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF