Switch to useEffect approach for unread marker fix

Revert the skeleton guard change and instead fix the useEffect in
ReportActionsList to also fire when reportLastReadTime transitions
from empty to populated, so the marker initializes correctly after
sign-in without showing an extra skeleton flash.

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

Author: MelvinBot

src/pages/inbox/report/ReportActionsList.tsx

@@ -262,7 +262,7 @@ function ReportActionsList({
         setUnreadMarkerTime(reportLastReadTime);
 
         // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, [report.reportID]);
+    }, [report.reportID, reportLastReadTime === '' ? 'empty' : 'populated']);
 
     const prevUnreadMarkerReportActionID = useRef<string | null>(null);
 

src/pages/inbox/report/ReportActionsView.tsx

@@ -368,8 +368,7 @@ function ReportActionsView({reportID, onLayout}: ReportActionsViewProps) {
     const shouldShowSkeletonForInitialLoad = isLoadingInitialReportActions && (isReportDataIncomplete || isMissingReportActions) && !isOffline;
 
     // Show skeleton while the app is loading and we're online
-    // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
-    const shouldShowSkeletonForAppLoad = (isLoadingApp || (isLoadingInitialReportActions && !hasOnceLoadedReportActions)) && !isOffline;
+    const shouldShowSkeletonForAppLoad = isLoadingApp && !isOffline;
 
     // Show skeleton for the Concierge side panel until report data has been
     // loaded at least once. Before the first openReport response, hasOlderActions