Project

General

Profile

Development #31326

Faciliter la surcharge du sujet des couriels dans les intégrations graphiques

Added by Emmanuel Cazenave 4 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Start date:
12 Mar 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Dans il y a templates/email/subject.txt :

{% block subject-prefix %}{% autoescape off %}{% firstof global_title site_name site "Compte Citoyen" %}{% endautoescape %} :{% endblock %} {% block email-subject %}{% endblock %}

On gagnerait à inclure l'espace après ':' dans le bloc subject-prefix pour qu'un espace ne traîne pas au début du sujet si on veut se passer de subject-prefix.

0001-templates-include-space-in-subjet-prefix-block-31326.patch View (885 Bytes) Emmanuel Cazenave, 12 Mar 2019 04:35 PM

0001-templates-try-a-new-variable-in-subjet-prefix-block-.patch View (910 Bytes) Emmanuel Cazenave, 12 Mar 2019 05:03 PM

Associated revisions

Revision d625c6e4 (diff)
Added by Emmanuel Cazenave 2 months ago

templates: try a new variable in subjet-prefix block (#31326)

History

#1 Updated by Emmanuel Cazenave 4 months ago

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

Les villes ont quand même souvent envie quand même d'un préfixe, plutôt que devoir l'ajouter à chacun des messages définis dans les workflows. Ok pour ce patch mais le vrai truc souhaité ce ne serait pas avoir une variable supplémentaire en premier du {% firstof .... %} ?

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

Et ça s'appellerait genre global_email_prefix et ça serait défini via l'écran de paramétrage des emails dans Hobo.

#4 Updated by Emmanuel Cazenave 4 months ago

Avec global_email_prefix.

Je le pousserai bien maintenant, ce qui me permettrait le petit patch qui va bien dans rouen-publik-theme, où la demande est bien de revenir à l'état initial, une absence de préfixe dans les mails de wcs (#31228).

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

Mmm, non. Ça laissera un comportement cohérent à tous les messages, et donc ça modifiera les messages d'authentic pour ne plus contenir de préfixe.

Ce que tu veux, c'est taper templates/qommon/email_subject.txt avec juste {{ subject }} comme contenu, côté Rouen.

#6 Updated by Emmanuel Cazenave 4 months ago

Frédéric Péters a écrit :

Mmm, non. Ça laissera un comportement cohérent à tous les messages, et donc ça modifiera les messages d'authentic pour ne plus contenir de préfixe.

Pas compris.

Ce que tu veux, c'est taper templates/qommon/email_subject.txt avec juste {{ subject }} comme contenu, côté Rouen.

C'est bien le template que je pensais surcharger, mais j'imaginais quelque chose d'inutilement compliqué (l'héritage c'est comme les antibiotiques, c'est pas automatique) :

{% extends "emails/subject.txt" %}
{% block subject-prefix %}{% endblock %}
{% block email-subject %}{{ subject }}{% endblock %}

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

Pas compris.

Aujourd'hui, de tout temps, les emails envoyés par Authentic sont préfixés du nom; comme je t'imaginais encore actif sur templates/emails/subject.txt, je pointais que ça affecterait également Authentic.

Bref, on a donc ici aucune urgence côté Rouen, vu que tu peux taper templates/qommon/email_subject.txt dedans ?

#8 Updated by Emmanuel Cazenave 4 months ago

Frédéric Péters a écrit :

Bref, on a donc ici aucune urgence côté Rouen, vu que tu peux taper templates/qommon/email_subject.txt dedans ?

Tout à fait, désolé pour ce dialogue laborieux.

#9 Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution proposée to En cours
  • Assignee set to Emmanuel Cazenave

Finalement, est-ce qu'on veut ce changement ou pas ?

#10 Updated by Emmanuel Cazenave 3 months ago

  • Status changed from En cours to Solution proposée

Oui oui le dernier patch permet la définition d'un préfixe global dans le sujet des mails.

#11 Updated by Benjamin Dauvergne 3 months ago

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

Ack alors.

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

  • Status changed from Solution validée to Résolu (à déployer)
commit d625c6e444e50acc9dba51b305b5a55b594dba97
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Tue Mar 12 16:33:12 2019 +0100

    templates: try a new variable in subjet-prefix block (#31326)

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

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

Also available in: Atom PDF