Checkpoint: ✅ Modification de la boîte de dialogue de test de rappels

- Ajout d'un champ de saisie pour l'email du destinataire
- Modification du texte : "Tous les rappels configurés pour cette séquence seront envoyés à l'adresse email que vous indiquez ci-dessous"
- Validation de l'email avant envoi
- Backend modifié pour envoyer à l'email saisi au lieu de tous les inscrits
- Les logs enregistrent correctement l'email de test utilisé

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}
               >

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',

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