Merge pull request Expensify#43250 from bernhardoj/fix/43120-openrepo…

…rt-twice-when-open-thread Fix opening a thread calls OpenReport twice
nkdengineer · Jun 10, 2024 · cd77d59 · cd77d59
2 parents bb10b8b + f801dc8
commit cd77d59
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/src/components/ReportActionItem/MoneyRequestAction.tsx b/src/components/ReportActionItem/MoneyRequestAction.tsx
@@ -10,7 +10,6 @@ import * as IOUUtils from '@libs/IOUUtils';
 import Navigation from '@libs/Navigation/Navigation';
 import * as ReportActionsUtils from '@libs/ReportActionsUtils';
 import type {ContextMenuAnchor} from '@pages/home/report/ContextMenu/ReportActionContextMenu';
-import * as Report from '@userActions/Report';
 import CONST from '@src/CONST';
 import type {TranslationPaths} from '@src/languages/types';
 import ONYXKEYS from '@src/ONYXKEYS';
@@ -92,7 +91,6 @@ function MoneyRequestAction({
         }
 
         const childReportID = action?.childReportID ?? '0';
-        Report.openReport(childReportID);
         Navigation.navigate(ROUTES.REPORT_WITH_ID.getRoute(childReportID));
     };
 

diff --git a/src/libs/actions/Report.ts b/src/libs/actions/Report.ts
@@ -1019,7 +1019,6 @@ function navigateToAndOpenReportWithAccountIDs(participantAccountIDs: number[])
  */
 function navigateToAndOpenChildReport(childReportID = '0', parentReportAction: Partial<ReportAction> = {}, parentReportID = '0') {
     if (childReportID !== '0') {
-        openReport(childReportID);
         Navigation.navigate(ROUTES.REPORT_WITH_ID.getRoute(childReportID));
     } else {
         const participantAccountIDs = [...new Set([currentUserAccountID, Number(parentReportAction.actorAccountID)])];