Développement #13552
Ne pas recalculer le template du message à chaque envoi
0%
Description
Au lieu de compiler le template pour chaque abonné en y inserant le lien de désabonnement, le corps du message doit être calculé une seule fois et le lien est remplacé pour chaque abonné.
Files
Associated revisions
History
Updated by Serghei Mihai about 8 years ago
- File 0001-compute-email-message-body-once-13552.patch added
- Status changed from Nouveau to En cours
- Assignee set to Serghei Mihai
- Patch proposed changed from No to Yes
Updated by Serghei Mihai about 8 years ago
- File deleted (
0001-compute-email-message-body-once-13552.patch)
Updated by Serghei Mihai about 8 years ago
- File 0001-compute-email-message-body-once-13552.patch 0001-compute-email-message-body-once-13552.patch added
Patch à jour avec le placeholder défini directement dans le fichier modèle.
Updated by Serghei Mihai about 8 years ago
Chez moi ils passent, mais je suis preneur d'un ack ou nack de qqun qui a appliqué le patch en local et qui l'a executé.
Updated by Frédéric Péters about 8 years ago
J'étendrais le test_unsubscription_link pour vérifier que les messages contiennent bien chacun des unsubscription_link différents.
Updated by Serghei Mihai about 8 years ago
- File 0001-compute-email-message-body-once-13552.patch 0001-compute-email-message-body-once-13552.patch added
Ok
Updated by Frédéric Péters about 8 years ago
Il manque pas un niveau d'indentation sur la dernière ligne ?
(et j'aurais plutôt fait ça avec un dictionnaire, assert link not in links / links[link] = True.)
Updated by Serghei Mihai about 8 years ago
Non, car le lien de desabonnement est fait par catégorie et non par annonce.
Updated by Serghei Mihai about 8 years ago
- Status changed from En cours to Résolu (à déployer)
commit 9841eeb472deae48c898b3462629bc208435844a
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Tue Oct 4 17:03:28 2016 +0200
compute email message body once (#13552)
compute email message body once (#13552)