From 94fed67d1908d66070666e4fb29d785848469ae1 Mon Sep 17 00:00:00 2001
From: Manus <dev-agent@manus.ai>
Date: Thu, 22 Jan 2026 08:33:07 -0500
Subject: [PATCH] =?UTF-8?q?Checkpoint:=20=E2=9C=85=20Modification=20de=20l?=
 =?UTF-8?q?a=20bo=C3=AEte=20de=20dialogue=20de=20test=20de=20rappels=20-?=
 =?UTF-8?q?=20Ajout=20d'un=20champ=20de=20saisie=20pour=20l'email=20du=20d?=
 =?UTF-8?q?estinataire=20-=20Modification=20du=20texte=20:=20"Tous=20les?=
 =?UTF-8?q?=20rappels=20configur=C3=A9s=20pour=20cette=20s=C3=A9quence=20s?=
 =?UTF-8?q?eront=20envoy=C3=A9s=20=C3=A0=20l'adresse=20email=20que=20vous?=
 =?UTF-8?q?=20indiquez=20ci-dessous"=20-=20Validation=20de=20l'email=20ava?=
 =?UTF-8?q?nt=20envoi=20-=20Backend=20modifi=C3=A9=20pour=20envoyer=20?=
 =?UTF-8?q?=C3=A0=20l'email=20saisi=20au=20lieu=20de=20tous=20les=20inscri?=
 =?UTF-8?q?ts=20-=20Les=20logs=20enregistrent=20correctement=20l'email=20d?=
 =?UTF-8?q?e=20test=20utilis=C3=A9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 client/src/pages/AdminSequences.tsx | 33 +++++++++++++++++++++++++----
 server/routers.ts                   | 24 +++++++++++++--------
 todo.md                             |  1 +
 3 files changed, 45 insertions(+), 13 deletions(-)

diff --git a/client/src/pages/AdminSequences.tsx b/client/src/pages/AdminSequences.tsx
index 83145df..ef5a935 100644
--- a/client/src/pages/AdminSequences.tsx
+++ b/client/src/pages/AdminSequences.tsx
@@ -35,6 +35,7 @@ export default function AdminSequences() {
   const [isCompactView, setIsCompactView] = useState(true);
   const [isConfirmTestOpen, setIsConfirmTestOpen] = useState(false);
   const [sequenceToTest, setSequenceToTest] = useState<number | null>(null);
+  const [testEmail, setTestEmail] = useState("");
 
   const [formData, setFormData] = useState({
     formationId: "",
@@ -94,17 +95,20 @@ export default function AdminSequences() {
       toast.success("Email de test envoyé avec succès");
       setIsConfirmTestOpen(false);
       setSequenceToTest(null);
+      setTestEmail("");
     },
     onError: (error) => {
       toast.error(`Erreur lors de l'envoi du test : ${error.message}`);
       setIsConfirmTestOpen(false);
       setSequenceToTest(null);
+      setTestEmail("");
     },
   });
 
   const handleTestRappel = (sequenceId: number, sequence: any) => {
     if (sequence.dates && sequence.dates.length > 0) {
       setSequenceToTest(sequenceId);
+      setTestEmail(""); // Réinitialiser l'email
       setIsConfirmTestOpen(true);
     } else {
       toast.error("Cette séquence n'a pas de dates configurées");
@@ -113,7 +117,14 @@ export default function AdminSequences() {
 
   const confirmTestRappel = () => {
     if (sequenceToTest !== null) {
-      testerRappelMutation.mutate({ sequenceId: sequenceToTest });
+      if (!testEmail || !testEmail.includes('@')) {
+        toast.error("Veuillez saisir une adresse email valide");
+        return;
+      }
+      testerRappelMutation.mutate({ 
+        sequenceId: sequenceToTest,
+        emailTest: testEmail 
+      });
     }
   };
 
@@ -1076,17 +1087,31 @@ export default function AdminSequences() {
         <Dialog open={isConfirmTestOpen} onOpenChange={setIsConfirmTestOpen}>
           <DialogContent>
             <DialogHeader>
-              <DialogTitle>Confirmer l'envoi du test</DialogTitle>
+              <DialogTitle>Tester l'envoi de rappels</DialogTitle>
               <DialogDescription>
-                Êtes-vous sûr de vouloir envoyer un email de test de rappel à tous les apprenants inscrits confirmés de cette séquence ?
+                Tous les rappels configurés pour cette séquence seront envoyés à l'adresse email que vous indiquez ci-dessous.
               </DialogDescription>
             </DialogHeader>
-            <div className="flex justify-end gap-2 pt-4">
+            <div className="space-y-4 py-4">
+              <div className="space-y-2">
+                <Label htmlFor="test-email">Adresse email du destinataire</Label>
+                <Input
+                  id="test-email"
+                  type="email"
+                  placeholder="exemple@domaine.com"
+                  value={testEmail}
+                  onChange={(e) => setTestEmail(e.target.value)}
+                  disabled={testerRappelMutation.isPending}
+                />
+              </div>
+            </div>
+            <div className="flex justify-end gap-2">
               <Button 
                 variant="outline" 
                 onClick={() => {
                   setIsConfirmTestOpen(false);
                   setSequenceToTest(null);
+                  setTestEmail("");
                 }}
                 disabled={testerRappelMutation.isPending}
               >
diff --git a/server/routers.ts b/server/routers.ts
index 0499bdd..23c37c2 100644
--- a/server/routers.ts
+++ b/server/routers.ts
@@ -1418,6 +1418,7 @@ export const appRouter = router({
     testerRappel: adminProcedure
       .input(z.object({
         sequenceId: z.number(),
+        emailTest: z.string().email().optional(),
       }))
       .mutation(async ({ input }) => {
         const sequence = await db.getSequenceById(input.sequenceId);
@@ -1463,26 +1464,31 @@ export const appRouter = router({
           throw new TRPCError({ code: 'BAD_REQUEST', message: 'Aucun apprenant inscrit confirmé pour cette séquence' });
         }
 
-        // Envoyer le rappel de test à l'administrateur connecté
-           // Envoyer le rappel de test à tous les inscrits confirmés
+        // Envoyer le rappel de test
         const { sendRappelEmail } = await import('./emailService');
 
         let nbEnvoyes = 0;
         let nbEchecs = 0;
         const erreurs: string[] = [];
 
-        for (const inscrit of inscritsConfirmes) {
-          const apprenant = inscrit.apprenant;
-          
-          if (!apprenant || !apprenant.email) {
+        // Si un email de test est fourni, envoyer à cet email uniquement
+        // Sinon, envoyer à tous les inscrits confirmés
+        const destinataires = input.emailTest 
+          ? [{ apprenant: inscritsConfirmes[0].apprenant, emailOverride: input.emailTest }]
+          : inscritsConfirmes.map(i => ({ apprenant: i.apprenant, emailOverride: null }));
+
+        for (const { apprenant, emailOverride } of destinataires) {
+          if (!apprenant || (!apprenant.email && !emailOverride)) {
             nbEchecs++;
             erreurs.push(`Apprenant ${apprenant?.nom || 'inconnu'}: email manquant`);
             continue;
           }
 
+          const emailDestinataire = emailOverride || apprenant.email;
+
           try {
             await sendRappelEmail({
-              apprenantEmail: apprenant.email,
+              apprenantEmail: emailDestinataire,
               apprenantPrenom: apprenant.prenom,
               apprenantNom: apprenant.nom,
               apprenantFonction: apprenant.fonction || '',
@@ -1506,7 +1512,7 @@ export const appRouter = router({
               rappelId: rappelAUtiliser.id,
               sequenceId: input.sequenceId,
               apprenantId: apprenant.id,
-              email: apprenant.email,
+              email: emailDestinataire,
               type: rappelAUtiliser.templateType,
               typeEnvoi: 'test',
               statut: 'succes',
@@ -1524,7 +1530,7 @@ export const appRouter = router({
                 rappelId: rappelAUtiliser.id,
                 sequenceId: input.sequenceId,
                 apprenantId: apprenant.id,
-                email: apprenant.email,
+                email: emailDestinataire,
                 type: rappelAUtiliser.templateType,
                 typeEnvoi: 'test',
                 statut: 'echec',
diff --git a/todo.md b/todo.md
index d2a4fb9..61204c4 100644
--- a/todo.md
+++ b/todo.md
@@ -1188,3 +1188,4 @@
 
 - [x] Corriger l'enregistrement des logs de rappels (table logsrappels reste vide après test)
 - [x] Ajouter une boîte de dialogue de confirmation avant l'envoi de test des rappels
+- [x] Modifier la boîte de dialogue de test de rappels pour permettre de saisir l'email du destinataire