Bug Fixes in Snapshot Policy: Schedule Editing and Index Expression D…

…isplay (#1207) (#1208) (cherry picked from commit 657a699) Signed-off-by: Sandeep Kumawat <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Sandeep Kumawat <[email protected]> Co-authored-by: Ramakrishna Chilaka <[email protected]>
opensearch-project · Dec 16, 2024 · 94236a1 · 94236a1
1 parent d8c01a4
commit 94236a1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/public/pages/CreateSnapshotPolicy/components/CronSchedule/CronSchedule.tsx b/public/pages/CreateSnapshotPolicy/components/CronSchedule/CronSchedule.tsx
@@ -88,20 +88,20 @@ const CronSchedule = ({
 
   function onDayOfWeekChange(dayOfWeek: string) {
     setWeek(dayOfWeek);
-    // changeCron({ dayOfWeek });
+    changeCron({ dayOfWeek });
   }
 
   function onDayOfMonthChange(dayOfMonth: number) {
     setMonth(dayOfMonth);
-    // changeCron({ dayOfMonth });
+    changeCron({ dayOfMonth });
   }
 
   function onStartTimeChange(date: moment.Moment) {
     const minute = date.minute();
     const hour = date.hour();
     setMinute(minute);
     setHour(hour);
-    // changeCron({ minute, hour });
+    changeCron({ minute, hour });
   }
 
   function onTypeChange(e: ChangeEvent<HTMLSelectElement>) {

diff --git a/public/pages/CreateSnapshotPolicy/containers/CreateSnapshotPolicy/CreateSnapshotPolicy.tsx b/public/pages/CreateSnapshotPolicy/containers/CreateSnapshotPolicy/CreateSnapshotPolicy.tsx
@@ -477,7 +477,7 @@ export class CreateSnapshotPolicy extends MDSEnabledComponent<CreateSMPolicyProp
     const selectedIndexOptions = [...this.state.selectedIndexOptions, newOption];
     this.setState({
       selectedIndexOptions: selectedIndexOptions,
-      policy: this.setPolicyHelper("snapshot_config.indices", selectedIndexOptions.toString()),
+      policy: this.setPolicyHelper("snapshot_config.indices", selectedIndexOptions.map((option) => option.label).join(", ")),
     });
   };