fix(schedules): don’t display warning when viewing only schedule

sws2apps · Sep 29, 2023 · 73a9990 · 73a9990
1 parent 74a944b
commit 73a9990
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/features/schedules/PersonSelect.jsx b/src/features/schedules/PersonSelect.jsx
@@ -249,13 +249,13 @@ const PersonSelect = ({
   useEffect(() => {
     setHasWarning(false);
 
-    if (selectedPerson !== null) {
+    if (selectedPerson !== null && edit) {
       const activeRecords = Schedules.history.filter(
         (record) => record.weekOf === currentWeek && record.studentID === selectedPerson.person_uid
       );
       setHasWarning(activeRecords.length > 1);
     }
-  }, [assID, selectedPerson, currentWeek, refreshCurrent]);
+  }, [assID, selectedPerson, currentWeek, refreshCurrent, edit]);
 
   return (
     <HtmlTooltip