Merge pull request #184 from ITPNYU/hotfix/update_safetytrained

Call fetchIsSafetyTrained when a user's email address is updated
ITPNYU · Apr 15, 2024 · 5c8df94 · 5c8df94
2 parents 5c07b3e + 6620007
commit 5c8df94
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/media_commons_booking_app/src/client/routes/booking/bookingProvider.tsx b/media_commons_booking_app/src/client/routes/booking/bookingProvider.tsx
@@ -45,7 +45,7 @@ export function BookingProvider({ children }) {
   const [bookingCalendarInfo, setBookingCalendarInfo] =
     useState<DateSelectArg>();
   const [department, setDepartment] = useState<Department>();
-  const [isSafetyTrained, setIsSafetyTrained] = useState(true);
+  const [isSafetyTrained, setIsSafetyTrained] = useState(false);
   const [role, setRole] = useState<Role>();
   const [selectedRooms, setSelectedRooms] = useState<RoomSetting[]>([]);
 
@@ -56,10 +56,6 @@ export function BookingProvider({ children }) {
       .includes(userEmail);
   }, [userEmail, bannedUsers]);
 
-  useEffect(() => {
-    fetchIsSafetyTrained();
-  }, []);
-
   const fetchIsSafetyTrained = useCallback(async () => {
     if (!userEmail) return;
     let isTrained = safetyTrainedUsers
@@ -74,6 +70,10 @@ export function BookingProvider({ children }) {
     setIsSafetyTrained(isTrained);
   }, [userEmail, safetyTrainedUsers]);
 
+  useEffect(() => {
+    fetchIsSafetyTrained();
+  }, [fetchIsSafetyTrained]);
+
   return (
     <BookingContext.Provider
       value={{