Fix: use usePrevious for empty-to-value transition instead of reportLastReadTime dependency

Reverts the reportLastReadTime dependency array change that caused a regression
(unread marker not showing for new messages). Now uses usePrevious to detect
only the empty-to-non-empty transition of reportLastReadTime, preserving the
original reportID-only dependency for the main effect.

Co-authored-by: Pujan Shah <Pujan92@users.noreply.github.com>

Author: MelvinBot

src/pages/inbox/report/ReportActionsList.tsx

@@ -248,6 +248,7 @@ function ReportActionsList({
     const prevSortedVisibleReportActionsObjects = usePrevious(sortedVisibleReportActionsObjects);
 
     const reportLastReadTime = report.lastReadTime ?? '';
+    const prevReportLastReadTime = usePrevious(reportLastReadTime);
 
     /**
      * The timestamp for the unread marker.
@@ -259,15 +260,19 @@ function ReportActionsList({
      */
     const [unreadMarkerTime, setUnreadMarkerTime] = useState(reportLastReadTime);
     useEffect(() => {
-        // When lastReadTime is empty (e.g. data hasn't loaded yet after sign-in),
-        // skip the update so we don't place the marker at the wrong position.
-        if (reportLastReadTime === '') {
-            return;
-        }
         setUnreadMarkerTime(reportLastReadTime);
 
         // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, [report.reportID, reportLastReadTime]);
+    }, [report.reportID]);
+
+    useEffect(() => {
+        // Only update when lastReadTime transitions from empty to a real value
+        // (e.g. after sign-in when report data loads for the first time).
+        if (prevReportLastReadTime !== '' || reportLastReadTime === '') {
+            return;
+        }
+        setUnreadMarkerTime(reportLastReadTime);
+    }, [prevReportLastReadTime, reportLastReadTime]);
 
     const prevUnreadMarkerReportActionID = useRef<string | null>(null);