Development #44411
emails, paramétrage explicite pour le libellé du From
0%
Description
Aujourd'hui on prend le titre général donné à la plateforme, et c'est une bonne valeur par défaut, mais on pourrait vouloir autre chose.
Paramètre à ajouter à l'écran + modification au settings loader pour l'utiliser, autour de là :
if variables.get('global_title'):
tenant_settings.DEFAULT_FROM_EMAIL = '"%s" <%s>' % (
variables['global_title'].replace('"', ' '), variables['default_from_email'])
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a presque 4 ans
- Statut changé de Nouveau à Solution proposée
J'espère ne pas tomber hors sujet : j'ai prévu le cas où sur l'IHM, un libellé est renseigné alors que l'adresse From ne l'est pas.
(pour ne pas à avoir à expliquer sur l'IHM que dans ce cas le champs libellé ne s'appliquerait pas).
Mis à jour par Frédéric Péters il y a presque 4 ans
help_text copié/collé/modifié erroné.
Houlala regex, non, j'ai même pas l'énergie de chercher à comprendre le cas que ça essaie de couvrir; c'est pour te couvrir s'il y a des déploiements existants qui écrivaient "blah <adresse@domaine>" dans le paramétrage du from ?
Ça existe ?
Ça peut pas juste être corrigé et garder le code ici simple ?
Mis à jour par Nicolas Roche il y a presque 4 ans
- Fichier 0001-emails-allow-user-to-define-default-from-mail-label-.patch 0001-emails-allow-user-to-define-default-from-mail-label-.patch ajouté
- Patch proposed changé de Non à Oui
help_text erroné.
Je propose : 'Default display name for email sender (defaults to plateform title)'
et du coup je change aussi le nom de la variable : 'default_sender_name'
c'est pour te couvrir s'il y a des déploiements existants qui écrivaient "blah <adresse@domaine>" dans le paramétrage du from ?
Ça peut pas juste être corrigé et garder le code ici simple ?
Oui, je n'avais pas vu que si l'ajout du "display sender name" était
conditionné par l'existence de la variable default_from_email, c'était
justement pour éviter ce cas.
Mis à jour par Frédéric Péters il y a presque 4 ans
Default display name
Si tu parles de "par défaut" ça fait imaginer qu'il pourrait y en avoir un autre, alors que nom.
Je mettrais juste "Sender name".
Mis à jour par Nicolas Roche il y a presque 4 ans
- Nom du champ :
Sender name - Texte d'aide :
(defaults to plateform title) - Nom de la variable :
email_sender_name
Mis à jour par Nicolas Roche il y a presque 4 ans
Mis à jour par Thomas Noël il y a presque 4 ans
(Au passage, tu peux ajouter email_sender_name dans la (nouvelle) liste des variables "auto" qui s'appelle AUTO_VARIABLES dans hobo.environment.models) (tu devras rebaser pour la voir, c'est nouveau, #44325)
Mis à jour par Nicolas Roche il y a presque 4 ans
- Fichier 0001-emails-allow-user-to-define-email-sender-name-44411.patch 0001-emails-allow-user-to-define-email-sender-name-44411.patch ajouté
(ajout de email_sender_name dans AUTO_VARIABLES)
Mis à jour par Thomas Noël il y a presque 4 ans
Pourquoi avoir supprimé la ligne « settings.ALLOWED_SPF_RECORDS = ['include:allowed_mx.com'] » des tests ? Ca me semble pas bien (ça retire le test sur SPF).
Par ailleurs il faudra un patch dans wcs pour prendre en charge cette nouvelle variable (wcs n'utilise pas le settings_loader de hobo) : ça se passe dans la gestion de msg['From'] dans wcs/qommon/emails.py (ticket à faire dans wcs pour parler de ça).
Enfin, je suis pas très sûr de "(defaults to plateform title)" parce que jamais on ne parle de "plateform" dans hobo. Restons plutôt sur le terme global title : "(defaults to global title)"
Mis à jour par Nicolas Roche il y a presque 4 ans
- Lié à Bug #44565: modifier le texte d'aide du préfix des mails ajouté
Mis à jour par Nicolas Roche il y a presque 4 ans
- Lié à Development #44566: Prendre en charge la variable hobo "email_sender_name" ajouté
Mis à jour par Nicolas Roche il y a presque 4 ans
- Fichier 0001-emails-allow-user-to-define-email-sender-name-44411.patch 0001-emails-allow-user-to-define-email-sender-name-44411.patch ajouté
Pourquoi avoir supprimé la ligne « settings.ALLOWED_SPF_RECORDS = ['include:allowed_mx.com'] » des tests ?
Je pensais que c'était hors sujet (puisque traité par les tests spécifiques au dessus) et que ça résultait d'un mauvais copier/coller de ma part (lors de #41430).
Mais j'aurais du mieux nettoyer et faire un ticket dédié, désolé.
Mis à jour par Thomas Noël il y a presque 4 ans
Je suis pas super chaud pour le help_text, j'aurais préféré un truc genre « Nom complet de l'expéditeur (si absent, le titre général est utilisé) ». Et pour différencier du default_from_email, préciser qu'il faut ici indiquer une adresse email, avec un help_text « Adresse email par défaut de l'expéditeur »
(traductions en anglais laissées en exercice au lecteur...)
Mis à jour par Nicolas Roche il y a presque 4 ans
- Fichier 0001-emails-allow-user-to-define-email-sender-name-44411.patch 0001-emails-allow-user-to-define-email-sender-name-44411.patch ajouté
Voici le détail de la partie retravaillée :
default_from_email = ValidEmailField(label=_('Default From'),
help_text=_('Custom sender email address'))
email_sender_name = forms.CharField(label=_('Sender Name'), required=False,
help_text=_('Custom sender name (defaults to global title)'))
global_email_prefix = forms.CharField(label=_('Prefix'), required=False,
help_text=_('Custom prefix for emails subject (defaults to global title)'))
Mis à jour par Thomas Noël il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Ok avec :
Custom sender email address → Sender email addressMis à jour par Nicolas Roche il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 02052aff40c4188fb807b9d8601ae606e7458b6b
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Thu Jun 25 18:49:26 2020 +0200
emails: allow user to define email sender name (#44411)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
emails: allow user to define email sender name (#44411)