Fix update metrics

grafana · Jun 4, 2024 · 63bc4d0 · 63bc4d0
1 parent 2ab8315
commit 63bc4d0
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/engine/apps/alerts/tasks/notify_user.py b/engine/apps/alerts/tasks/notify_user.py
@@ -254,9 +254,6 @@ def notify_user_task(
                             notification_channel=notification_policy.notify_by,
                         )
 
-        if log_record:  # log_record is None if user notification policy step is unspecified
-            log_record.save()
-
         is_notification_triggered = (
             log_record.type == UserNotificationPolicyLogRecord.TYPE_PERSONAL_NOTIFICATION_TRIGGERED
             if log_record
@@ -273,6 +270,9 @@ def notify_user_task(
         ):
             update_metrics_for_user.apply_async((user.id,))
 
+        if log_record:  # log_record is None if user notification policy step is unspecified
+            log_record.save()
+
         if not stop_escalation:
             # if the step is NOTIFY and notification was not not bundled, perform regular notification
             # and update time when user was notified