Merge pull request #177 from monzo/limit-hours

Limit the closing incident notification to working hours
monzo · Nov 5, 2019 · 094ccd6 · 094ccd6
2 parents ed1f72f + 9a6cea4
commit 094ccd6
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/response/slack/incident_notifications.py b/response/slack/incident_notifications.py
@@ -36,6 +36,10 @@ def remind_close_incident(incident: Incident):
     if datetime.now().weekday() in (5, 6):
         return
 
+    # Only remind during the day to prevent alerting people at unsociable hours
+    if datetime.now().hour not in range(9, 18):
+        return
+
     try:
         comms_channel = CommsChannel.objects.get(incident=incident)
         if not incident.is_closed():