From 18c30a9ee6d84a53b854e5ae0c44d9bba2804c04 Mon Sep 17 00:00:00 2001
From: Michelle Fu <michelle.fu@sentry.io>
Date: Wed, 28 Aug 2024 10:30:26 -0700
Subject: [PATCH] set resolve threshold to null if alert is dynamic

---
 static/app/views/alerts/rules/metric/ruleForm.tsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/static/app/views/alerts/rules/metric/ruleForm.tsx b/static/app/views/alerts/rules/metric/ruleForm.tsx
index b0bb7d2d304c9a..c585f6754292d3 100644
--- a/static/app/views/alerts/rules/metric/ruleForm.tsx
+++ b/static/app/views/alerts/rules/metric/ruleForm.tsx
@@ -806,7 +806,11 @@ class RuleFormContainer extends DeprecatedAsyncComponent<Props, State> {
             ...activatedAlertFields,
             projects: [project.slug],
             triggers: sanitizedTriggers,
-            resolveThreshold: isEmpty(resolveThreshold) ? null : resolveThreshold,
+            resolveThreshold:
+              isEmpty(resolveThreshold) ||
+              detectionType === AlertRuleComparisonType.DYNAMIC
+                ? null
+                : resolveThreshold,
             thresholdType,
             thresholdPeriod,
             comparisonDelta: comparisonDelta ?? null,