put this back

Author: mifu67

src/sentry/incidents/subscription_processor.py

@@ -376,10 +376,6 @@ def process_update(self, subscription_update: QuerySubscriptionUpdate) -> None:
 
         aggregation_value = self.get_aggregation_value(subscription_update, comparison_delta)
 
-        if aggregation_value is None:
-            metrics.incr("incidents.alert_rules.skipping_update_invalid_aggregation_value")
-            return
-
         if aggregation_value is not None:
             if has_metric_alert_processing:
                 if self.alert_rule.detection_type == AlertRuleDetectionType.DYNAMIC:
@@ -450,6 +446,10 @@ def process_update(self, subscription_update: QuerySubscriptionUpdate) -> None:
             if potential_anomalies is None:
                 return
 
+        if aggregation_value is None:
+            metrics.incr("incidents.alert_rules.skipping_update_invalid_aggregation_value")
+            return
+
         fired_incident_triggers: list[IncidentTrigger] = []
         with transaction.atomic(router.db_for_write(AlertRule)):
             # Triggers is the threshold - NOT an instance of a trigger