fix: notification summary

don't count failed notifications that have reached max attempts as Pending
flanksource · Sep 17, 2024 · 79dd66d · 79dd66d
1 parent 339c791
commit 79dd66d
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/views/015_job_history.sql b/views/015_job_history.sql
@@ -237,7 +237,8 @@ FROM
   LEFT JOIN notification_send_summary ON notifications.id = notification_send_summary.notification_id
   LEFT JOIN event_queue ON 
     notifications.id::TEXT = event_queue.properties->>'notification_id' AND
-    event_queue.name = 'notification.send'
+    event_queue.name = 'notification.send' AND
+    event_queue.attempts < 4
 WHERE
   notifications.deleted_at IS NULL
 GROUP BY notifications.id,