feat: Modification de la détection des envois précédents pour les mai…

…ls de notification et de mise à jour des listes de candidats
mission-apprentissage · Jan 8, 2025 · 41356f1 · 41356f1
1 parent 7868540
commit 41356f1
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 7 deletions.
diff --git a/server/src/jobs/sendNotificationEmails.js b/server/src/jobs/sendNotificationEmails.js
@@ -69,15 +69,15 @@ async function sendNotificationEmails({ sendEmail, resendEmail }, options = {})
       const user = delegue ?? responsable;
 
       if (!user) {
-        stats.skiped++;
         logger.error("Utilisateur introuvable pour la relation " + relation._id);
+        stats.skiped++;
         return stats;
         // throw Error("Utilisateur introuvable pour la relation " + relation._id);
       }
 
       if (!user.email) {
-        stats.skiped++;
         logger.error("Absence d'adresse courriel pour l'utilisateur " + user._id);
+        stats.skiped++;
         return stats;
         // throw Error("Absence d'adresse courriel pour l'utilisateur " + user._id);
       }
@@ -86,7 +86,10 @@ async function sendNotificationEmails({ sendEmail, resendEmail }, options = {})
 
       const templateName = `notification_${type}`;
       const previous = user.emails.find(
-        (e) => e.templateName === templateName && e.data?.relation?._id === relation._id
+        (e) =>
+          e.templateName === templateName &&
+          e.data?.relation?._id === relation._id &&
+          e.data?.relation.last_date_voeux === relation.last_date_voeux
       );
 
       stats.total++;

diff --git a/server/src/jobs/sendUpdateEmails.js b/server/src/jobs/sendUpdateEmails.js
@@ -67,18 +67,28 @@ async function sendUpdateEmails({ sendEmail, resendEmail }, options = {}) {
       const user = delegue ?? responsable;
 
       if (!user) {
-        throw Error("Utilisateur introuvable pour la relation " + relation._id);
+        logger.error("Utilisateur introuvable pour la relation " + relation._id);
+        stats.skiped++;
+        return stats;
+        // throw Error("Utilisateur introuvable pour la relation " + relation._id);
       }
 
       if (!user.email) {
-        // return stats;
-        throw Error("Absence d'adresse courriel pour l'utilisateur " + user._id);
+        logger.error("Absence d'adresse courriel pour l'utilisateur " + user._id);
+        stats.skiped++;
+        return stats;
+        // throw Error("Absence d'adresse courriel pour l'utilisateur " + user._id);
       }
 
       const type = (delegue ? DownloadType.DELEGUE : DownloadType.RESPONSABLE).toLowerCase();
 
       const templateName = `notification_${type}`;
-      const previous = user.emails.find((e) => e.templateName === templateName && e.data.relation._id === relation._id);
+      const previous = user.emails.find(
+        (e) =>
+          e.templateName === templateName &&
+          e.data.relation?._id === relation._id &&
+          e.data.relation?.last_date_voeux === relation.last_date_voeux
+      );
       stats.total++;
 
       try {