Added caching to load all data at the start

Author: shashank-madan

src/components/Reports/TotalReport/TotalContributorsReport.jsx

@@ -17,7 +17,15 @@ function TotalContributorsReport({ startDate, endDate, userProfiles, darkMode, u
 
   const fromDate = useMemo(() => startDate.toLocaleDateString('en-CA'), [startDate]);
   const toDate = useMemo(() => endDate.toLocaleDateString('en-CA'), [endDate]);
-  const userList = useMemo(() => userProfiles.map(({ _id }) => _id), [userProfiles]);
+  const userList = useMemo(() => {
+    const list = userProfiles?.map(({ _id }) => _id) || [];
+    // eslint-disable-next-line no-console
+    console.log('TotalContributorsReport userList created:', {
+      userProfilesLength: userProfiles?.length,
+      userListLength: list.length,
+    });
+    return list;
+  }, [userProfiles]);
 
   // Fetch time entries for the selected period
   const loadTimeEntriesForPeriod = useCallback(async (controller) => {
@@ -26,12 +34,59 @@ function TotalContributorsReport({ startDate, endDate, userProfiles, darkMode, u
     if (!url) {
       return;
     }
+
+    // Don't make API call if userList is empty
+    if (!userList || userList.length === 0) {
+      // eslint-disable-next-line no-console
+      console.warn('TotalContributorsReport: Skipping API call - userList is empty', {
+        userProfilesLength: userProfiles?.length,
+        userListLength: userList?.length,
+      });
+      setTimeEntries([]);
+      setLoading(false);
+      return;
+    }
+
+    // Check cache with date range key to ensure cache is valid for current date range
+    const cacheKey = `TotalContributorsReport_${fromDate}_${toDate}`;
+    const cachedData = localStorage.getItem(cacheKey);
+    if (cachedData) {
+      try {
+        const parsedData = JSON.parse(cachedData);
+        if (parsedData && Array.isArray(parsedData) && parsedData.length > 0) {
+          // eslint-disable-next-line no-console
+          console.log('TotalContributorsReport: Using cached data', {
+            cacheKey,
+            dataLength: parsedData.length,
+          });
+          setTimeEntries(parsedData);
+          setLoading(false);
+          return;
+        }
+      } catch (e) {
+        // eslint-disable-next-line no-console
+        console.warn('TotalContributorsReport: Failed to parse cached data', e);
+      }
+    }
+
     try {
+      // eslint-disable-next-line no-console
+      console.log('TotalContributorsReport API Request:', {
+        url,
+        payload: { users: userList, fromDate, toDate },
+        usersCount: userList?.length,
+        timestamp: new Date().toISOString(),
+      });
       const response = await axios.post(
         url,
         { users: userList, fromDate, toDate },
         { signal: controller.signal }
       );
+      // eslint-disable-next-line no-console
+      console.log('TotalContributorsReport API Response:', {
+        dataLength: response.data?.length,
+        timestamp: new Date().toISOString(),
+      });
       const mappedTimeEntries = response.data.map(entry => ({
         userId: entry.personId,
         hours: entry.hours,
@@ -40,14 +95,26 @@ function TotalContributorsReport({ startDate, endDate, userProfiles, darkMode, u
         date: entry.dateOfWork,
       }));
       setTimeEntries(mappedTimeEntries);
+      
+      // Cache the data with date range key
+      if (mappedTimeEntries.length > 0) {
+        localStorage.setItem(cacheKey, JSON.stringify(mappedTimeEntries));
+        // eslint-disable-next-line no-console
+        console.log('TotalContributorsReport: Data cached', { cacheKey, dataLength: mappedTimeEntries.length });
+      } else {
+        // eslint-disable-next-line no-console
+        console.warn('TotalContributorsReport: Empty response - clearing cache', { cacheKey });
+        localStorage.removeItem(cacheKey);
+      }
     } catch (error) {
       // eslint-disable-next-line import/no-named-as-default-member
       if (!axios.isCancel(error)) {
-        // Handle error silently or show user-friendly message
+        // eslint-disable-next-line no-console
+        console.error('TotalContributorsReport API Error:', error);
         setTimeEntries([]);
       }
     }
-  }, [fromDate, toDate, userList]);
+  }, [fromDate, toDate, userList, userProfiles]);
 
   // Group time entries by user and calculate total hours
   const sumByUser = useCallback((entries) => {
@@ -141,13 +208,23 @@ function TotalContributorsReport({ startDate, endDate, userProfiles, darkMode, u
 
   // Load data when date range changes
   useEffect(() => {
+    // Only make API call if userList has data
+    if (!userList || userList.length === 0) {
+      // eslint-disable-next-line no-console
+      console.log('TotalContributorsReport: Waiting for userProfiles to load...', {
+        userProfilesLength: userProfiles?.length,
+        userListLength: userList?.length,
+      });
+      return;
+    }
+
     setLoading(true);
     const controller = new AbortController();
     loadTimeEntriesForPeriod(controller).then(() => {
       setLoading(false);
     });
     return () => controller.abort();
-  }, [loadTimeEntriesForPeriod]);
+  }, [loadTimeEntriesForPeriod, userList]);
 
   // Process data when time entries are loaded
   useEffect(() => {

src/components/Reports/TotalReport/TotalPeopleReport.jsx

@@ -57,13 +57,37 @@ function TotalPeopleReport(props) {
         return;
       }
 
+      // Check cache with date range key to ensure cache is valid for current date range
+      const cacheKey = `TotalPeopleReport_${fromDate}_${toDate}`;
+      const cachedData = localStorage.getItem(cacheKey);
+      if (cachedData) {
+        try {
+          const parsedData = JSON.parse(cachedData);
+          if (parsedData && Array.isArray(parsedData) && parsedData.length > 0) {
+            // eslint-disable-next-line no-console
+            console.log('TotalPeopleReport: Using cached data', {
+              cacheKey,
+              dataLength: parsedData.length,
+            });
+            setAllTimeEntries(parsedData);
+            setTotalPeopleReportDataLoading(false);
+            setTotalPeopleReportDataReady(true);
+            return;
+          }
+        } catch (e) {
+          // eslint-disable-next-line no-console
+          console.warn('TotalPeopleReport: Failed to parse cached data', e);
+        }
+      }
+
       try {
         // eslint-disable-next-line no-console
         console.log('TotalPeopleReport API Request:', {
           url,
           payload: { users: userList, fromDate, toDate },
           usersCount: userList?.length,
           userProfilesCount: userProfiles?.length,
+          timestamp: new Date().toISOString(),
         });
         const res = await axios.post(
           url,
@@ -74,22 +98,48 @@ function TotalPeopleReport(props) {
         console.log('TotalPeopleReport API Response:', {
           dataLength: res.data?.length,
           sampleData: res.data?.slice(0, 2),
+          timestamp: new Date().toISOString(),
+          responseTime: new Date().toISOString(),
         });
+        
         const timeEntries = res.data.map(entry => ({
           userId: entry.personId,
           hours: entry.hours,
           minutes: entry.minutes,
           isTangible: entry.isTangible,
           date: entry.dateOfWork,
         }));
+        
+        // Log if response seems incomplete (very few entries for many users)
+        if (timeEntries.length > 0 && userList.length > 100 && timeEntries.length < 10) {
+          // eslint-disable-next-line no-console
+          console.warn('TotalPeopleReport: Response may be incomplete', {
+            timeEntriesCount: timeEntries.length,
+            usersCount: userList.length,
+            ratio: (timeEntries.length / userList.length * 100).toFixed(2) + '%',
+            message: 'If data seems incomplete, backend may still be processing. Try refreshing in a few minutes.',
+          });
+        }
+        
         setAllTimeEntries(timeEntries);
+        
+        // Cache the data with date range key
+        if (timeEntries.length > 0) {
+          localStorage.setItem(cacheKey, JSON.stringify(timeEntries));
+          // eslint-disable-next-line no-console
+          console.log('TotalPeopleReport: Data cached', { cacheKey, dataLength: timeEntries.length });
+        } else {
+          // eslint-disable-next-line no-console
+          console.warn('TotalPeopleReport: Empty response - clearing cache', { cacheKey });
+          localStorage.removeItem(cacheKey);
+        }
       } catch (error) {
         // eslint-disable-next-line no-console
         console.error('TotalPeopleReport API Error:', error);
         setTotalPeopleReportDataLoading(false);
       }
     },
-    [fromDate, toDate, userList],
+    [fromDate, toDate, userList, userProfiles],
   );
 
   const sumByUser = useCallback((objectArray, property) => {