Merge pull request #730 from getmaxun/schedule-ui

fix: schedule modal ui update

Author: amhsirak

src/components/robot/Recordings.tsx

@@ -14,7 +14,7 @@ import { useTranslation } from "react-i18next";
 interface RecordingsProps {
   handleEditRecording: (id: string, fileName: string) => void;
   handleRunRecording: (settings: RunSettings) => void;
-  handleScheduleRecording: (settings: ScheduleSettings) => void;
+  handleScheduleRecording: (settings: ScheduleSettings) => Promise<boolean>;
   setRecordingInfo: (id: string, name: string) => void;
 }
 

src/components/robot/ScheduleSettings.tsx

@@ -10,7 +10,7 @@ import { getSchedule, deleteSchedule } from '../../api/storage';
 
 interface ScheduleSettingsProps {
   isOpen: boolean;
-  handleStart: (settings: ScheduleSettings) => void;
+  handleStart: (settings: ScheduleSettings) => Promise<boolean>;
   handleClose: () => void;
   initialSettings?: ScheduleSettings | null;
 }
@@ -272,7 +272,12 @@ export const ScheduleSettingsModal = ({ isOpen, handleStart, handleClose, initia
                 </Dropdown>
               </Box>
               <Box mt={2} display="flex" justifyContent="flex-end">
-                <Button onClick={() => handleStart(settings)} variant="contained" color="primary">
+                <Button onClick={async () => {
+                  const success = await handleStart(settings);
+                  if (success) {
+                    await getRobotSchedule();
+                  }
+                }} variant="contained" color="primary">
                   {t('schedule_settings.buttons.save_schedule')}
                 </Button>
                 <Button

src/pages/MainPage.tsx

@@ -235,15 +235,14 @@ export const MainPage = ({ handleEditRecording, initialContent }: MainPageProps)
     }
   }, [runningRecordingName, sockets, ids, debugMessageHandler, user?.id, t, notify, setRerenderRuns, setQueuedRuns, navigate, setContent, setIds]);
 
-  const handleScheduleRecording = (settings: ScheduleSettings) => {
-    scheduleStoredRecording(runningRecordingId, settings)
-      .then(({ message, runId }: ScheduleRunResponse) => {
-        if (message === 'success') {
-          notify('success', t('main_page.notifications.schedule_success', { name: runningRecordingName }));
-        } else {
-          notify('error', t('main_page.notifications.schedule_failed', { name: runningRecordingName }));
-        }
-      });
+  const handleScheduleRecording = async (settings: ScheduleSettings) => {
+    const { message, runId }: ScheduleRunResponse = await scheduleStoredRecording(runningRecordingId, settings);
+    if (message === 'success') {
+      notify('success', t('main_page.notifications.schedule_success', { name: runningRecordingName }));
+    } else {
+      notify('error', t('main_page.notifications.schedule_failed', { name: runningRecordingName }));
+    }
+    return message === 'success';
   }
 
   const DisplayContent = () => {