'fix' mypy by ignorning it. We should eventually get this consistent and use the data conditions to store the grouping rules.

Author: saponifi3d

src/sentry/workflow_engine/endpoints/validators/error_detector.py

@@ -16,9 +16,11 @@
 class ErrorDetectorValidator(BaseDetectorTypeValidator):
     data_source_required = False
 
+    # TODO - Update the Error Detector to store grouping rules in the DataCondition
     condition_group = serializers.DictField(
-        required=False
-    )  # This is only used to validate that it's empty
+        required=False,
+        allow_null=True,
+    )  # type: ignore[assignment]  # Intentionally overrides nested serializer to accept empty dicts
 
     fingerprinting_rules = serializers.CharField(required=False, allow_blank=True, allow_null=True)
 
@@ -41,11 +43,7 @@ def validate_condition_group(self, value: Any) -> Any:
                 "Condition group is not supported for error detectors"
             )
 
-        if value is not None:
-            # We want to allow an empty DataConditionGroup, but not create anything.
-            return None
-
-        return value
+        return None
 
     def validate_name(self, value: Any) -> str:
         # if name is different from existing, raise an error