Bug Fixes in Snapshot Policy: Schedule Editing and Index Expression Display

Signed-off-by: Sandeep Kumawat <[email protected]>

Author: Sandeep Kumawat

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>) {

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(", ")),
     });
   };
 
@@ -1002,6 +1002,10 @@ export class CreateSnapshotPolicy extends MDSEnabledComponent<CreateSMPolicyProp
     this.setState({ policy: this.setPolicyHelper("snapshot_config.partial", e.target.checked) });
   };
 
+  onIndicesToggle = (e: ChangeEvent<HTMLInputElement>) => {
+    this.setState({ policy: this.setPolicyHelper("snapshot_config.partial", e.target.checked) });
+  };
+
   setPolicyHelper = (path: string, newValue: any) => {
     return _.set(this.state.policy, path, newValue);
   };