edit correlation Alert Trigger fix (#1180)

* edit correlation Alert Trigger fix Signed-off-by: Riya Saxena <[email protected]> * delete correlation alert trigger fix Signed-off-by: Riya Saxena <[email protected]> * delete correlation alert trigger fix Signed-off-by: Riya Saxena <[email protected]> * edit correlation rule trigger fix Signed-off-by: Riya Saxena <[email protected]> * edit correlation rule trigger fix Signed-off-by: Riya Saxena <[email protected]> * edit correlation rule trigger fix Signed-off-by: Riya Saxena <[email protected]> * extra spacing removed Signed-off-by: Riya Saxena <[email protected]> --------- Signed-off-by: Riya Saxena <[email protected]> (cherry picked from commit 8220c38) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Oct 10, 2024 · 7e402d0 · 7e402d0
1 parent bfadcf2
commit 7e402d0
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/public/pages/Correlations/containers/CreateCorrelationRule.tsx b/public/pages/Correlations/containers/CreateCorrelationRule.tsx
@@ -243,7 +243,6 @@ export const CreateCorrelationRule: React.FC<CreateCorrelationRuleProps> = (
           setInitialValues(ruleRes);
         }
       };
-
       setAction('Edit');
       setInitialRuleValues();
     }
@@ -1064,7 +1063,11 @@ export const CreateCorrelationRule: React.FC<CreateCorrelationRuleProps> = (
                             data-test-subj="delete-alert-trigger-icon"
                             iconType="trash"
                             color="danger"
-                            onClick={() => setShowForm(false)}
+                            onClick={() => {
+                              // Clear the trigger values by setting them to an empty object or your initial state
+                              props.setFieldValue('trigger', {});
+                              setShowForm(false);
+                            }}
                           />
                         </EuiFormRow>
                       </EuiFlexItem>

diff --git a/public/store/CorrelationsStore.ts b/public/store/CorrelationsStore.ts
@@ -148,6 +148,7 @@ export class CorrelationsStore implements ICorrelationsStore {
         name: hit._source.name,
         time_window: hit._source.time_window || 300000,
         queries,
+        trigger: hit._source?.trigger
       };
     }
 
@@ -173,6 +174,7 @@ export class CorrelationsStore implements ICorrelationsStore {
           name: hit._source.name,
           time_window: hit._source.time_window || 300000,
           queries,
+          trigger: hit._source?.trigger
         };
       });
     }

diff --git a/types/Correlations.ts b/types/Correlations.ts
@@ -54,7 +54,7 @@ export interface CorrelationRuleModel {
   name: string;
   time_window: number; // Time in milliseconds
   queries: CorrelationRuleQuery[];
-  trigger: CorrelationRuleTrigger | undefined;
+  trigger?: CorrelationRuleTrigger;
 }
 
 export interface CorrelationRule extends CorrelationRuleModel {
@@ -76,7 +76,7 @@ export interface CorrelationRuleSource {
   name: string;
   time_window: number;
   correlate: CorrelationRuleSourceQueries[];
-  trigger?: CorrelationRuleTrigger | undefined;
+  trigger?: CorrelationRuleTrigger;
 }
 
 export interface CorrelationRuleHit {