don't display report as one-transaction report if it has previously d…

…eleted IOU requests
dominictb · Apr 4, 2024 · f9d5eaa · f9d5eaa
1 parent 853f71b
commit f9d5eaa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/libs/ReportActionsUtils.ts b/src/libs/ReportActionsUtils.ts
@@ -230,7 +230,8 @@ function getOneTransactionThreadReportID(reportActions: OnyxEntry<ReportActions>
             action.actionName === CONST.REPORT.ACTIONS.TYPE.IOU &&
             (iouRequestTypes.includes(action.originalMessage.type) ?? []) &&
             action.childReportID &&
-            action.originalMessage.IOUTransactionID,
+            // Include deleted IOU reportActions because they might have associated comments that we'd want to display
+            (action.originalMessage.deleted || action.originalMessage.IOUTransactionID),
     );
 
     // If we don't have any IOU request actions, or we have more than one IOU request actions, this isn't a oneTransaction report

diff --git a/src/types/onyx/OriginalMessage.ts b/src/types/onyx/OriginalMessage.ts
@@ -66,6 +66,7 @@ type IOUMessage = {
     type: ValueOf<typeof CONST.IOU.REPORT_ACTION_TYPE>;
     cancellationReason?: string;
     paymentType?: PaymentMethodType;
+    deleted?: string;
     /** Only exists when we are sending money */
     IOUDetails?: IOUDetails;
 };