Projet

Général

Profil

Development #13552

Ne pas recalculer le template du message à chaque envoi

Ajouté par Serghei Mihai il y a plus de 7 ans. Mis à jour il y a environ 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Serghei Mihai
Version cible:
-
Début:
11 octobre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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é.

Fichiers

0001-compute-email-message-body-once-13552.patch (3,42 ko) 0001-compute-email-message-body-once-13552.patch Serghei Mihai, 11 octobre 2016 17:16
0001-compute-email-message-body-once-13552.patch (4,89 ko) 0001-compute-email-message-body-once-13552.patch Serghei Mihai, 13 octobre 2016 18:03

Révisions associées

Révision 9841eeb4 (diff)
Ajouté par Serghei Mihai il y a plus de 7 ans

compute email message body once (#13552)

Historique

#1

Mis à jour par Serghei Mihai il y a plus de 7 ans

  • Fichier 0001-compute-email-message-body-once-13552.patch ajouté
  • Statut changé de Nouveau à En cours
  • Assigné à mis à Serghei Mihai
  • Patch proposed changé de Non à Oui
#2

Mis à jour par Serghei Mihai il y a plus de 7 ans

  • Fichier 0001-compute-email-message-body-once-13552.patch supprimé
#3

Mis à jour par Serghei Mihai il y a plus de 7 ans

Patch à jour avec le placeholder défini directement dans le fichier modèle.

#4

Mis à jour par Benjamin Dauvergne il y a plus de 7 ans

Si ça passe les tests, ack.

#5

Mis à jour par Serghei Mihai il y a plus de 7 ans

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é.

#6

Mis à jour par Frédéric Péters il y a plus de 7 ans

J'étendrais le test_unsubscription_link pour vérifier que les messages contiennent bien chacun des unsubscription_link différents.

#8

Mis à jour par Frédéric Péters il y a plus de 7 ans

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.)

#9

Mis à jour par Serghei Mihai il y a plus de 7 ans

Non, car le lien de desabonnement est fait par catégorie et non par annonce.

#10

Mis à jour par Frédéric Péters il y a plus de 7 ans

Ok, testé.

#11

Mis à jour par Serghei Mihai il y a plus de 7 ans

  • Statut changé de En cours à 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)
#12

Mis à jour par Serghei Mihai il y a environ 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF