fix: move the set markerLastReadTime outside the loop func

Signed-off-by: dominictb <[email protected]>
Expensify · Jun 10, 2024 · cc42221 · cc42221
1 parent 5941b95
commit cc42221
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/pages/home/report/ReportActionsList.tsx b/src/pages/home/report/ReportActionsList.tsx
@@ -514,13 +514,15 @@ function ReportActionsList({
             markerFound = true;
             if (!currentUnreadMarker || currentUnreadMarker !== reportAction.reportActionID) {
                 cacheUnreadMarkers.set(report.reportID, reportAction.reportActionID);
-                if (!markerLastReadTimeRef.current) {
-                    markerLastReadTimeRef.current = lastReadTimeRef.current;
-                }
                 setCurrentUnreadMarker(reportAction.reportActionID);
             }
         });
 
+        // if marker can be found, set the markerLastReadTimeRef to the last read time if necessary
+        if (markerFound && !markerLastReadTimeRef.current) {
+            markerLastReadTimeRef.current = lastReadTimeRef.current;
+        }
+
         if (!markerFound && !linkedReportActionID) {
             setCurrentUnreadMarker(null);
         }