Bug #23138
lingo: absence des template vars dans le contexte des courriels des nouvelles factures
0%
Description
Les templates des mails font référence à site_title
et portal_url
or ses variables n'existent pas dans le contexte
Fichiers
Demandes liées
Révisions associées
lingo: do not mention "Combo" in user messages (#23138)
tests: do not depend on message strings (#23138)
Historique
Mis à jour par Serghei Mihai il y a environ 6 ans
- Fichier 0001-lingo-update-new-invoices-email-s-context-with-templ.patch 0001-lingo-update-new-invoices-email-s-context-with-templ.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 6 ans
ctx = {'item': invoice, 'payment_url': urlparse.urljoin(settings.SITE_BASE_URL, payment_url)}
Je sais que le "item" était déjà là avant mais arrêtons et appelons ça invoice.
with override_settings(LANGUAGE_CODE='fr', TEMPLATE_VARS={'site_title': 'User Portal', 'portal_url': 'http://example.net'}):
Un peu la misère quand même d'avoir de manière non documentée l'exigence de variables; et je relis #13122 et trouve à me citer : « Ces variables : site_title & portal_url n'existent pas sans hobo. ». Clairement je serais pour que Combo fournisse des templates qui n'exigent pas hobo.
Mis à jour par Serghei Mihai il y a environ 6 ans
- Fichier 0001-lingo-update-new-invoices-email-s-context-with-templ.patch 0001-lingo-update-new-invoices-email-s-context-with-templ.patch ajouté
- Assigné à mis à Serghei Mihai
Ok, plus de item
.
Mis à jour par Serghei Mihai il y a environ 6 ans
- Lié à Development #23195: lingo: mettre à jour les templates des notifications des nouvelles factures ajouté
Mis à jour par Serghei Mihai il y a environ 6 ans
- Fichier 0001-lingo-update-new-invoices-email-s-context-with-templ.patch 0001-lingo-update-new-invoices-email-s-context-with-templ.patch ajouté
Et en faisant un test en local d'envoi d'un mail utilisant les templates du thème, je n'ai pas le nom du site, ni les couleurs du thème dans le mail.
Il manquaient les template vars dans le contexte.
Mis à jour par Thomas Noël il y a environ 6 ans
Mettre plus d'abord les templatevars dans le contexte (et finir par un update avec ce qu'on veut être sûr d'avoir, invoice, payment_url, tout ça).
Je m'interroge un poil, on peut pas utiliser plutôt un RequestContext qui jouera tous les context_processor, histoire d'être sûr de pas en rater ?
Mis à jour par Serghei Mihai il y a environ 6 ans
- Fichier 0001-lingo-update-new-invoices-email-s-context-with-templ.patch 0001-lingo-update-new-invoices-email-s-context-with-templ.patch ajouté
Pour RequestContext il faut avoir un request
, que je pourrais créer avec RequestFactory
, mais ça ne le passe pas par tous les context_processors.
Mis à jour par Serghei Mihai il y a environ 6 ans
- Fichier 0001-lingo-update-new-invoices-email-s-context-with-templ.patch 0001-lingo-update-new-invoices-email-s-context-with-templ.patch ajouté
Ouch la boulette.
Merci.
Mis à jour par Serghei Mihai il y a presque 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
commit 21c21969f20f9692c4a765cdf3d249760e378aff (origin/master) Author: Serghei Mihai <smihai@entrouvert.com> Date: Fri Apr 6 08:02:01 2018 +0200 lingo: update new invoices email's context with template vars (#23138)
Mis à jour par Frédéric Péters il y a presque 6 ans
Dans la suite, la traduction, échec puis échec jenkins, et maintenant combo qui n'expose jamais ce nom à l'usager va se trouver le mentionner dans les messages, pas top.
Mis à jour par Frédéric Péters il y a presque 6 ans
(avant de retagguer, dégager ce "sur combo" des trads)
Mis à jour par Serghei Mihai il y a presque 6 ans
- Statut changé de Résolu (à déployer) à En cours
Ok...
Mis à jour par Serghei Mihai il y a presque 6 ans
Mis à jour par Frédéric Péters il y a presque 6 ans
msgid "You can view it by going on your <a href=\"%(portal_url)s\">Combo</a>."
Ça dépasse la traduction.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de En cours à Résolu (à déployer)
Parce que pas tenté par à nouveau avoir une mise à jour qui arrive le mercredi après-midi, j'ai poussé des corrections et je vais pousser un tag et installer sur les recettes etc.
commit 4cd6169edfc9f07c2357bdc4f9860ffc9c539ac4 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 22 21:09:40 2018 +0200 translation update commit db5ad3c1c846183ac415c4eddd3f5f8c80175091 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 22 21:07:49 2018 +0200 lingo: do not mention "Combo" in user messages (#23138)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: update new invoices email's context with template vars (#23138)