add the day and week budget hours in checkSchedule effect dependency

tijlleenders · Nov 9, 2024 · a0aae54 · a0aae54
1 parent 2c5be17
commit a0aae54
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/src/components/GoalsComponents/GoalConfigModal/ConfigGoal.tsx b/src/components/GoalsComponents/GoalConfigModal/ConfigGoal.tsx
@@ -313,11 +313,13 @@ const ConfigGoal = ({ type, goal, mode }: { type: TGoalCategory; mode: TGoalConf
 
   const checkSchedule = async () => {
     setScheduleStatus("pending");
-    const schedulerOutput = await checkGoalSchedule(getFinalTags());
-    console.log("schedulerOutput", schedulerOutput);
-    const status = await checkSchedulingStatus(schedulerOutput, goal.id);
-    console.log("status", status);
-    setScheduleStatus(status);
+    try {
+      const schedulerOutput = await checkGoalSchedule(getFinalTags());
+      const status = await checkSchedulingStatus(schedulerOutput, goal.id);
+      setScheduleStatus(status);
+    } catch (error) {
+      setScheduleStatus(null);
+    }
   };
 
   useEffect(() => {
@@ -327,7 +329,7 @@ const ConfigGoal = ({ type, goal, mode }: { type: TGoalCategory; mode: TGoalConf
     }, 1000);
 
     return () => clearTimeout(debounceTimer);
-  }, [tags.duration, afterTime, beforeTime, tags.on]);
+  }, [tags.duration, afterTime, beforeTime, perDayHrs, perWeekHrs]);
 
   return (
     <ZModal

diff --git a/src/hooks/useScheduler.tsx b/src/hooks/useScheduler.tsx
@@ -111,7 +111,7 @@ function useScheduler() {
       return res;
     } catch (error) {
       console.log("Error checking goal schedule:", error);
-      return undefined;
+      return null;
     }
   };