Repository navigation
Conversation
…s déjà partagés - Les campagnes déjà envoyées à Mailchimp sont maintenant cliquables depuis la liste des campagnes et affichent le mail en lecture seule (avec un lien vers l'archive Mailchimp quand elle existe). - Dans l'aperçu d'édition d'une campagne, les liens déjà présents dans une campagne envoyée sont signalés par un⚠️ suivi de la/des date(s) de publication. La comparaison ignore le schéma, le "www.", le slash final, l'ancre et la casse. Le badge n'apparaît jamais dans le mail envoyé. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…tagé Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
L'aperçu est recalculé à chaque modification de la campagne en cours. La détection chargeait toutes les campagnes envoyées en entités puis les convertissait une à une avec Valinor : ~600 ms avec ~250 campagnes. On ne lit plus que la date d'envoi et le JSON brut des campagnes envoyées (requête scalaire, sans hydratation), et on décode le JSON pour n'en garder que les URLs : ~15 ms pour le même volume, avec les mêmes résultats. Ajoute un test d'intégration sur la vraie base de test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Korbeil
approved these changes
Nov 3, 2026
Korbeil
left a comment
Korbeil
left a comment
Member
There was a problem hiding this comment.
LGTM , merci pour cette amélioration !
…s veilles En haut de l'aperçu d'édition, un bloc liste les liens déjà partagés dans une veille envoyée. Chaque date ouvre la veille concernée (en lecture seule) dans un nouvel onglet. Le bloc est placé hors des zones éditables de l'aperçu : le calque « Editer » affiché au survol ne le recouvre pas, de simples liens suffisent, sans JS. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Author
|
Je viens de mettre à jour la dernière screenshot pour ce nouveau commit. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Veille : consultation des anciennes campagnes et signalement des liens déjà partagés
Consulter les campagnes envoyées
Dans la liste des campagnes, seule la campagne en cours était cliquable. Les campagnes déjà envoyées à Mailchimp le sont maintenant aussi : elles ouvrent une page en lecture seule (
/admin/techletter/show/{id}) qui affiche le mail, avec un lien vers l'archive Mailchimp quand elle existe.Signaler les liens déjà partagés
Dans l'aperçu d'édition d'une campagne :
⚠️ 21/10/2026, 08/10/2026).Détails :
http/https, lewww., le slash final, l'ancre et la casse.Performance
L'aperçu est recalculé à chaque modification de la campagne. Seuls l'id, la date et le JSON brut des campagnes envoyées sont lus (requête scalaire, sans hydratation d'entités ni Valinor), puis on ne garde que les URLs : ~15 ms avec ~250 campagnes, contre ~600 ms en convertissant chaque campagne en modèle.
Captures
Liens clickable pour voir les précédentes techletters:
Affichage d'une précédente techletter:
Warning sur les liens:
