Project

General

Profile

Actions
Copy link

Développement #13552

closed

Ne pas recalculer le template du message à chaque envoi

Added by Serghei Mihai over 9 years ago. Updated almost 9 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Serghei Mihai
Target version:
-
Start date:
11 October 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
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é.

Files

Download all files
0001-compute-email-message-body-once-13552.patch (3.42 KB) 0001-compute-email-message-body-once-13552.patch Serghei Mihai, 11 October 2016 05:16 PM
0001-compute-email-message-body-once-13552.patch (4.89 KB) 0001-compute-email-message-body-once-13552.patch Serghei Mihai, 13 October 2016 06:03 PM
Actions
Copy link
 #1

Updated by Serghei Mihai over 9 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
Actions
Copy link
 #2

Updated by Serghei Mihai over 9 years ago

  • File deleted (0001-compute-email-message-body-once-13552.patch)
Actions
Copy link
 #3

Updated by Serghei Mihai over 9 years ago

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

Actions
Copy link
 #4

Updated by Benjamin Dauvergne over 9 years ago

Si ça passe les tests, ack.

Actions
Copy link
 #5

Updated by Serghei Mihai over 9 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é.

Actions
Copy link
 #6

Updated by Frédéric Péters over 9 years ago

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

Actions
Copy link
 #8

Updated by Frédéric Péters over 9 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.)

Actions
Copy link
 #9

Updated by Serghei Mihai over 9 years ago

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

Actions
Copy link
 #10

Updated by Frédéric Péters over 9 years ago

Ok, testé.

Actions
Copy link
 #11

Updated by Serghei Mihai over 9 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)
Actions
Copy link
 #12

Updated by Serghei Mihai almost 9 years ago

  • Status changed from Résolu (à déployer) to Fermé
Actions
Copy link

Also available in: Atom PDF