From d726dcbfa13d8c8f1c0fe4177561dba2ce9b1a7a Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 15:51:19 -0600
Subject: [PATCH 01/49] add consts

---
 src/CONST.ts                                       | 3 +++
 src/components/SelectionList/Search/ActionCell.tsx | 3 +++
 src/libs/SearchUIUtils.ts                          | 1 +
 3 files changed, 7 insertions(+)

diff --git a/src/CONST.ts b/src/CONST.ts
index a3df980ff5dd..f5441e6f942d 100755
--- a/src/CONST.ts
+++ b/src/CONST.ts
@@ -5644,6 +5644,9 @@ const CONST = {
         ACTION_TYPES: {
             VIEW: 'view',
             REVIEW: 'review',
+            SUBMIT: 'submit',
+            APPROVE: 'approve',
+            PAY: 'pay',
             DONE: 'done',
             PAID: 'paid',
         },
diff --git a/src/components/SelectionList/Search/ActionCell.tsx b/src/components/SelectionList/Search/ActionCell.tsx
index faafa6159dc1..57db7e7d7969 100644
--- a/src/components/SelectionList/Search/ActionCell.tsx
+++ b/src/components/SelectionList/Search/ActionCell.tsx
@@ -15,6 +15,9 @@ import type {SearchTransactionAction} from '@src/types/onyx/SearchResults';
 const actionTranslationsMap: Record<SearchTransactionAction, TranslationPaths> = {
     view: 'common.view',
     review: 'common.review',
+    submit: 'common.submit',
+    approve: 'iou.approve',
+    pay: 'iou.pay',
     done: 'common.done',
     paid: 'iou.settledExpensify',
 };
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index a7ce065a6d23..ed865d6eac2d 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -223,6 +223,7 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
 
             return {
                 ...transactionItem,
+                action: 'submit',
                 from,
                 to,
                 formattedFrom,

From 2f4c9802966b65fc23163394222aaf8231216707 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 16:26:25 -0600
Subject: [PATCH 02/49] create getAction

---
 src/libs/SearchUIUtils.ts | 34 ++++++++++++++++++++++++++++++++--
 1 file changed, 32 insertions(+), 2 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index ed865d6eac2d..7ce8cfb44c0a 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -4,6 +4,7 @@ import ChatListItem from '@components/SelectionList/ChatListItem';
 import ReportListItem from '@components/SelectionList/Search/ReportListItem';
 import TransactionListItem from '@components/SelectionList/Search/TransactionListItem';
 import type {ListItem, ReportActionListItemType, ReportListItemType, TransactionListItemType} from '@components/SelectionList/types';
+import * as IOU from '@libs/actions/IOU';
 import * as Expensicons from '@src/components/Icon/Expensicons';
 import CONST from '@src/CONST';
 import type {TranslationPaths} from '@src/languages/types';
@@ -11,7 +12,7 @@ import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
 import type * as OnyxTypes from '@src/types/onyx';
 import type SearchResults from '@src/types/onyx/SearchResults';
-import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction} from '@src/types/onyx/SearchResults';
+import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction} from '@src/types/onyx/SearchResults';
 import * as CurrencyUtils from './CurrencyUtils';
 import DateUtils from './DateUtils';
 import {translateLocal} from './Localize';
@@ -223,7 +224,7 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
 
             return {
                 ...transactionItem,
-                action: 'submit',
+                action: getAction(data, key),
                 from,
                 to,
                 formattedFrom,
@@ -241,6 +242,35 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
         });
 }
 
+/**
+ * @private
+ * Returns the action that can be taken on a given transaction or report
+ *
+ * Do not use directly, use only via `getSections()` facade.
+ */
+function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTransactionAction {
+    if (!isTransactionEntry(key) && !isReportEntry(key)) {
+        return CONST.SEARCH.ACTION_TYPES.VIEW;
+    }
+
+    return CONST.SEARCH.ACTION_TYPES.VIEW;
+
+    // const report = isReportEntry(key) ? data[key] : data[data[key].reportID];
+    // const chatReport = isReportEntry(key) ? data[key].chatReportID
+    // iouReport
+    // chatReport
+    // policy
+    // transactions
+    // if (IOU.canIOUBePaid()) {
+    // pay
+    // approve
+    // submit
+    // review
+    // view
+    // done
+    // paid
+}
+
 /**
  * @private
  * Organizes data into List Sections for display, for the ReportActionListItemType of Search Results.

From 584a539d4ad69b2dc000765d87178a659931f264 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 16:49:39 -0600
Subject: [PATCH 03/49] create canReportBeSubmitted

---
 .../SelectionList/Search/ActionCell.tsx       |  4 +--
 src/libs/SearchUIUtils.ts                     | 26 +++++++++++++------
 src/libs/actions/IOU.ts                       | 21 +++++++++++++++
 3 files changed, 41 insertions(+), 10 deletions(-)

diff --git a/src/components/SelectionList/Search/ActionCell.tsx b/src/components/SelectionList/Search/ActionCell.tsx
index 57db7e7d7969..612f515791c0 100644
--- a/src/components/SelectionList/Search/ActionCell.tsx
+++ b/src/components/SelectionList/Search/ActionCell.tsx
@@ -80,7 +80,7 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
         ) : null;
     }
 
-    if (action === CONST.SEARCH.ACTION_TYPES.REVIEW) {
+    // if (action === CONST.SEARCH.ACTION_TYPES.REVIEW) {
         return (
             <Button
                 text={text}
@@ -90,7 +90,7 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
                 innerStyles={buttonInnerStyles}
             />
         );
-    }
+    // }
 }
 
 ActionCell.displayName = 'ActionCell';
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 7ce8cfb44c0a..831ef1f49285 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -253,15 +253,23 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTr
         return CONST.SEARCH.ACTION_TYPES.VIEW;
     }
 
-    return CONST.SEARCH.ACTION_TYPES.VIEW;
+    if (isTransactionEntry(key) && !data[key].isFromOneTransactionReport) {
+        return CONST.SEARCH.ACTION_TYPES.VIEW;
+    }
 
-    // const report = isReportEntry(key) ? data[key] : data[data[key].reportID];
-    // const chatReport = isReportEntry(key) ? data[key].chatReportID
-    // iouReport
-    // chatReport
-    // policy
-    // transactions
-    // if (IOU.canIOUBePaid()) {
+    const transaction = isTransactionEntry(key) ? data[key] : {};
+    const report = isTransactionEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.reportID}`] : data[key] ?? {};
+    const chatReport = chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report.chatReportID}`] ?? {};
+    const policy = data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.policyID}`] ?? {};
+    const allTransaction = isTransactionEntry(key) ? transaction : report.transactions;
+
+    if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
+        return CONST.SEARCH.ACTION_TYPES.PAY;
+    }
+
+    if (IOU.canApproveIOU(report, policy)) {
+        return CONST.SEARCH.ACTION_TYPES.APPROVE;
+    }
     // pay
     // approve
     // submit
@@ -269,6 +277,8 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTr
     // view
     // done
     // paid
+
+    return CONST.SEARCH.ACTION_TYPES.VIEW;
 }
 
 /**
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 7ce9b9dfb272..6a79625cf55f 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7104,6 +7104,26 @@ function canIOUBePaid(
     );
 }
 
+function canReportBeSubmitted(
+    moneyRequestReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report>,
+    policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>,
+    currentUserAccountID: number,
+    transaction?: OnyxTypes.Transaction,
+) {
+    const isDraft = ReportUtils.isOpenExpenseReport(moneyRequestReport);
+    const {reimbursableSpend} = ReportUtils.getMoneyRequestSpendBreakdown(moneyRequestReport);
+    const hasAllPendingRTERViolations = TransactionUtils.allHavePendingRTERViolation([transaction?.transactionID ?? '-1']);
+    const shouldShowBrokenConnectionViolation = TransactionUtils.shouldShowBrokenConnectionViolation(transaction?.transactionID ?? '-1', moneyRequestReport, policy);
+    const isAdmin = policy?.role === CONST.POLICY.ROLE.ADMIN;
+
+   return !!moneyRequestReport &&
+    isDraft &&
+    reimbursableSpend !== 0 &&
+    !hasAllPendingRTERViolations &&
+    !shouldShowBrokenConnectionViolation &&
+    (moneyRequestReport?.ownerAccountID === currentUserAccountID || isAdmin || moneyRequestReport?.managerID === currentUserAccountID);
+}
+
 function getIOUReportActionToApproveOrPay(chatReport: OnyxEntry<OnyxTypes.Report>, excludedIOUReportID: string): OnyxEntry<ReportAction> {
     const chatReportActions = allReportActions?.[`${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${chatReport?.reportID}`] ?? {};
 
@@ -8615,5 +8635,6 @@ export {
     updateLastLocationPermissionPrompt,
     resolveDuplicates,
     getIOUReportActionToApproveOrPay,
+    canReportBeSubmitted,
 };
 export type {GPSPoint as GpsPoint, IOURequestType};

From 7724e84475d3fce8ca012b558498baedd0abd695 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 16:50:47 -0600
Subject: [PATCH 04/49] fix lint

---
 .../SelectionList/Search/ActionCell.tsx        | 18 +++++++++---------
 src/libs/SearchUIUtils.ts                      |  2 +-
 src/libs/actions/IOU.ts                        | 14 ++++++++------
 3 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/src/components/SelectionList/Search/ActionCell.tsx b/src/components/SelectionList/Search/ActionCell.tsx
index 612f515791c0..275767ba6b68 100644
--- a/src/components/SelectionList/Search/ActionCell.tsx
+++ b/src/components/SelectionList/Search/ActionCell.tsx
@@ -81,15 +81,15 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
     }
 
     // if (action === CONST.SEARCH.ACTION_TYPES.REVIEW) {
-        return (
-            <Button
-                text={text}
-                onPress={goToItem}
-                small
-                style={[styles.w100]}
-                innerStyles={buttonInnerStyles}
-            />
-        );
+    return (
+        <Button
+            text={text}
+            onPress={goToItem}
+            small
+            style={[styles.w100]}
+            innerStyles={buttonInnerStyles}
+        />
+    );
     // }
 }
 
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 831ef1f49285..05b17488bbf3 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -259,7 +259,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTr
 
     const transaction = isTransactionEntry(key) ? data[key] : {};
     const report = isTransactionEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.reportID}`] : data[key] ?? {};
-    const chatReport = chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report.chatReportID}`] ?? {};
+    const chatReport = (chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report.chatReportID}`] ?? {});
     const policy = data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.policyID}`] ?? {};
     const allTransaction = isTransactionEntry(key) ? transaction : report.transactions;
 
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 6a79625cf55f..0a07184c0852 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7116,12 +7116,14 @@ function canReportBeSubmitted(
     const shouldShowBrokenConnectionViolation = TransactionUtils.shouldShowBrokenConnectionViolation(transaction?.transactionID ?? '-1', moneyRequestReport, policy);
     const isAdmin = policy?.role === CONST.POLICY.ROLE.ADMIN;
 
-   return !!moneyRequestReport &&
-    isDraft &&
-    reimbursableSpend !== 0 &&
-    !hasAllPendingRTERViolations &&
-    !shouldShowBrokenConnectionViolation &&
-    (moneyRequestReport?.ownerAccountID === currentUserAccountID || isAdmin || moneyRequestReport?.managerID === currentUserAccountID);
+    return (
+        !!moneyRequestReport &&
+        isDraft &&
+        reimbursableSpend !== 0 &&
+        !hasAllPendingRTERViolations &&
+        !shouldShowBrokenConnectionViolation &&
+        (moneyRequestReport?.ownerAccountID === currentUserAccountID || isAdmin || moneyRequestReport?.managerID === currentUserAccountID)
+    );
 }
 
 function getIOUReportActionToApproveOrPay(chatReport: OnyxEntry<OnyxTypes.Report>, excludedIOUReportID: string): OnyxEntry<ReportAction> {

From def5f27bdb496494e8bfce99a0f8b78df6b6ed99 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 17:08:27 -0600
Subject: [PATCH 05/49] add more logic

---
 src/libs/SearchUIUtils.ts | 35 ++++++++++++++++++++++++-----------
 1 file changed, 24 insertions(+), 11 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 05b17488bbf3..efdbf2a41d58 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -4,7 +4,6 @@ import ChatListItem from '@components/SelectionList/ChatListItem';
 import ReportListItem from '@components/SelectionList/Search/ReportListItem';
 import TransactionListItem from '@components/SelectionList/Search/TransactionListItem';
 import type {ListItem, ReportActionListItemType, ReportListItemType, TransactionListItemType} from '@components/SelectionList/types';
-import * as IOU from '@libs/actions/IOU';
 import * as Expensicons from '@src/components/Icon/Expensicons';
 import CONST from '@src/CONST';
 import type {TranslationPaths} from '@src/languages/types';
@@ -13,11 +12,13 @@ import ROUTES from '@src/ROUTES';
 import type * as OnyxTypes from '@src/types/onyx';
 import type SearchResults from '@src/types/onyx/SearchResults';
 import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction} from '@src/types/onyx/SearchResults';
+import * as IOU from './actions/IOU';
 import * as CurrencyUtils from './CurrencyUtils';
 import DateUtils from './DateUtils';
 import {translateLocal} from './Localize';
 import Navigation from './Navigation/Navigation';
 import * as ReportActionsUtils from './ReportActionsUtils';
+import * as ReportUtils from './ReportUtils';
 import * as TransactionUtils from './TransactionUtils';
 
 const columnNamesToSortingProperty = {
@@ -258,10 +259,20 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTr
     }
 
     const transaction = isTransactionEntry(key) ? data[key] : {};
-    const report = isTransactionEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.reportID}`] : data[key] ?? {};
-    const chatReport = (chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report.chatReportID}`] ?? {});
+    const report = isTransactionEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.reportID}`] ?? {} : data[key] ?? {};
+    const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report.chatReportID}`] ?? {};
     const policy = data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.policyID}`] ?? {};
-    const allTransaction = isTransactionEntry(key) ? transaction : report.transactions;
+    const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
+    const allViolations = Object.keys(data).filter(item => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
+    const allTransactions = isTransactionEntry(key) ? transaction : Object.keys(data).filter(item => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION)) ?? {};
+
+    if (ReportUtils.isSettled(report.reportID)) {
+        return CONST.SEARCH.ACTION_TYPES.PAID;
+    }
+
+    if (ReportUtils.isSettled(report.reportID)) {
+        return CONST.SEARCH.ACTION_TYPES.DONE;
+    }
 
     if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
         return CONST.SEARCH.ACTION_TYPES.PAY;
@@ -270,13 +281,15 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTr
     if (IOU.canApproveIOU(report, policy)) {
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
-    // pay
-    // approve
-    // submit
-    // review
-    // view
-    // done
-    // paid
+
+    if (IOU.canReportBeSubmitted(report, policy, 1, transaction)) {
+        return CONST.SEARCH.ACTION_TYPES.SUBMIT;
+    }
+
+    const hasViolations = isTransactionEntry(key) ? TransactionUtils.hasViolation(transaction.transactionID, violations) : ReportUtils.hasViolations(report.reportID, allViolations);
+    if (hasViolations) {
+        return CONST.SEARCH.ACTION_TYPES.REVIEW;
+    }
 
     return CONST.SEARCH.ACTION_TYPES.VIEW;
 }

From 2822472217d3d7db82a4adc35dd3b241a830a6f0 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 17:17:09 -0600
Subject: [PATCH 06/49] fix some ts

---
 src/libs/SearchUIUtils.ts | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index efdbf2a41d58..d105ece30656 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -13,6 +13,7 @@ import type * as OnyxTypes from '@src/types/onyx';
 import type SearchResults from '@src/types/onyx/SearchResults';
 import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
+import * as Report from './actions/Report';
 import * as CurrencyUtils from './CurrencyUtils';
 import DateUtils from './DateUtils';
 import {translateLocal} from './Localize';
@@ -213,6 +214,7 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
     const shouldShowMerchant = getShouldShowMerchant(data);
 
     const doesDataContainAPastYearTransaction = shouldShowYear(data);
+    const currentUserAccountID = Report.getCurrentUserAccountID();
 
     return Object.keys(data)
         .filter(isTransactionEntry)
@@ -225,7 +227,7 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
 
             return {
                 ...transactionItem,
-                action: getAction(data, key),
+                action: getAction(data, key, currentUserAccountID),
                 from,
                 to,
                 formattedFrom,
@@ -249,7 +251,7 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
  *
  * Do not use directly, use only via `getSections()` facade.
  */
-function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTransactionAction {
+function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUserAccountID: number): SearchTransactionAction {
     if (!isTransactionEntry(key) && !isReportEntry(key)) {
         return CONST.SEARCH.ACTION_TYPES.VIEW;
     }
@@ -258,18 +260,20 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTr
         return CONST.SEARCH.ACTION_TYPES.VIEW;
     }
 
-    const transaction = isTransactionEntry(key) ? data[key] : {};
-    const report = isTransactionEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.reportID}`] ?? {} : data[key] ?? {};
-    const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report.chatReportID}`] ?? {};
-    const policy = data[`${ONYXKEYS.COLLECTION.REPORT}${transaction.policyID}`] ?? {};
+    const transaction = isTransactionEntry(key) ? data[key] : null;
+    const report = transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key];
+    const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report?.chatReportID}`] ?? {};
+    const policy = data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.policyID}`] ?? {};
     const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
-    const allViolations = Object.keys(data).filter(item => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
-    const allTransactions = isTransactionEntry(key) ? transaction : Object.keys(data).filter(item => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION)) ?? {};
+    const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
+    const allTransactions = isTransactionEntry(key) ? transaction : Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION)) ?? {};
 
+    // TODO: update isSettled to take report object instead of connect to Onyx
     if (ReportUtils.isSettled(report.reportID)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
     }
 
+    // TODO: create function to determine if report is done
     if (ReportUtils.isSettled(report.reportID)) {
         return CONST.SEARCH.ACTION_TYPES.DONE;
     }
@@ -282,7 +286,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTr
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
 
-    if (IOU.canReportBeSubmitted(report, policy, 1, transaction)) {
+    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
         return CONST.SEARCH.ACTION_TYPES.SUBMIT;
     }
 

From 89b9c6b8f36999268b0997d8d98af4063e8dbc17 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 17:24:00 -0600
Subject: [PATCH 07/49] add types

---
 src/types/onyx/SearchResults.ts | 34 +++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index 98e3aa66fa00..886f6a00a0d6 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -107,6 +107,40 @@ type SearchReport = {
 
     /** The action that can be performed for the report */
     action?: SearchTransactionAction;
+
+    /** The type of chat if this is a chat report */
+    chatType?: ValueOf<typeof CONST.REPORT.CHAT_TYPE>;
+
+    /** Invoice room receiver data */
+    invoiceReceiver?: Record<string, string>;
+
+    /** Whether the report has a single transaction */
+    isOneTransactionReport?: boolean;
+
+    /** Whether the report is policyExpenseChat */
+    isPolicyExpenseChat?: boolean;
+
+    /** Whether the report is waiting on a bank account */
+    isWaitingOnBankAccount?: boolean;
+
+    /** If the report contains nonreimbursable expenses, send the nonreimbursable total */
+    nonReimbursableTotal?: number;
+
+    /** Account ID of the report owner */
+    ownerAccountID?: number;
+
+    /** The state that the report is currently in */
+    stateNum?: ValueOf<typeof CONST.REPORT.STATE_NUM>;
+
+    /** The status of the current report */
+    statusNum?: ValueOf<typeof CONST.REPORT.STATUS_NUM>;
+
+    /** For expense reports, this is the total amount requested */
+    unheldTotal?: number;
+
+    /** Whether the report is archived */
+    // eslint-disable-next-line @typescript-eslint/naming-convention
+    private_isArchived?: string;
 };
 
 /** Model of report action search result */

From a3302a82fecd1e564e7123995506c5192749d565 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 17:34:23 -0600
Subject: [PATCH 08/49] add more types

---
 src/libs/SearchUIUtils.ts |  4 ++--
 src/libs/actions/IOU.ts   | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index d105ece30656..6fdfd8233df3 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -269,12 +269,12 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     const allTransactions = isTransactionEntry(key) ? transaction : Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION)) ?? {};
 
     // TODO: update isSettled to take report object instead of connect to Onyx
-    if (ReportUtils.isSettled(report.reportID)) {
+    if (ReportUtils.isSettled(report?.reportID)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
     }
 
     // TODO: create function to determine if report is done
-    if (ReportUtils.isSettled(report.reportID)) {
+    if (ReportUtils.isSettled(report?.reportID)) {
         return CONST.SEARCH.ACTION_TYPES.DONE;
     }
 
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 0a07184c0852..08fccd65a426 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -66,7 +66,7 @@ import type {ErrorFields, Errors} from '@src/types/onyx/OnyxCommon';
 import type {PaymentMethodType} from '@src/types/onyx/OriginalMessage';
 import type ReportAction from '@src/types/onyx/ReportAction';
 import type {OnyxData} from '@src/types/onyx/Request';
-import type {SearchTransaction} from '@src/types/onyx/SearchResults';
+import type {SearchReport, SearchTransaction} from '@src/types/onyx/SearchResults';
 import type {Comment, Receipt, ReceiptSource, Routes, SplitShares, TransactionChanges, WaypointCollection} from '@src/types/onyx/Transaction';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import * as CachedPDFPaths from './CachedPDFPaths';
@@ -7016,7 +7016,7 @@ function sendMoneyWithWallet(report: OnyxEntry<OnyxTypes.Report>, amount: number
     Report.notifyNewAction(params.chatReportID, managerID);
 }
 
-function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report>, policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>) {
+function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport, policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>) {
     // Only expense reports can be approved
     const isPaidGroupPolicy = policy && PolicyUtils.isPaidGroupPolicy(policy);
     if (!isPaidGroupPolicy) {
@@ -7041,8 +7041,8 @@ function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report>,
 }
 
 function canIOUBePaid(
-    iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report>,
-    chatReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report>,
+    iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
+    chatReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
     policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>,
     transactions?: OnyxTypes.Transaction[],
     onlyShowPayElsewhere = false,
@@ -7105,7 +7105,7 @@ function canIOUBePaid(
 }
 
 function canReportBeSubmitted(
-    moneyRequestReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report>,
+    moneyRequestReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
     policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>,
     currentUserAccountID: number,
     transaction?: OnyxTypes.Transaction,

From d38592d505be6e80f6c697b4557dc9e075cb2828 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 17:41:34 -0600
Subject: [PATCH 09/49] create searchpolicy type

---
 src/libs/SearchUIUtils.ts       |  2 +-
 src/types/onyx/SearchResults.ts | 50 ++++++++++++++++++++++++++++++++-
 2 files changed, 50 insertions(+), 2 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 6fdfd8233df3..97c662da0521 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -11,7 +11,7 @@ import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
 import type * as OnyxTypes from '@src/types/onyx';
 import type SearchResults from '@src/types/onyx/SearchResults';
-import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction} from '@src/types/onyx/SearchResults';
+import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction, SearchPolicy} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
 import * as Report from './actions/Report';
 import * as CurrencyUtils from './CurrencyUtils';
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index 886f6a00a0d6..dda45454af00 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -176,6 +176,43 @@ type SearchReportAction = {
     reportID: string;
 };
 
+/** Model of policy search result */
+type SearchPolicy = {
+    /** The policy type */
+    type: ValueOf<typeof CONST.POLICY.TYPE>;
+
+    /** Whether the auto reporting is enabled */
+    autoReporting?: boolean;
+
+    /**
+     * The scheduled submit frequency set up on this policy.
+     * Note that manual does not exist in the DB and thus should not exist in Onyx, only as a param for the API.
+     * "manual" really means "immediate" (aka "daily") && harvesting.enabled === false
+     */
+    autoReportingFrequency?: Exclude<ValueOf<typeof CONST.POLICY.AUTO_REPORTING_FREQUENCIES>, typeof CONST.POLICY.AUTO_REPORTING_FREQUENCIES.MANUAL>;
+
+    /** The approval mode set up on this policy */
+    approvalMode?: ValueOf<typeof CONST.POLICY.APPROVAL_MODE>;
+
+    /** The reimbursement choice for policy */
+    reimbursementChoice?: ValueOf<typeof CONST.POLICY.REIMBURSEMENT_CHOICES>;
+
+    /** The maximum report total allowed to trigger auto reimbursement */
+    autoReimbursementLimit?: number;
+
+    /** The verified bank account linked to the policy */
+    achAccount?: Record<string, string>;
+
+    /** The current user's role in the policy */
+    role: ValueOf<typeof CONST.POLICY.ROLE>;
+
+    /** The employee list of the policy */
+    employeeList?: Record<string, Record<string, string>>;
+
+    /** Detailed settings for the autoReimbursement */
+    autoReimbursement?: Record<string, string>;
+};
+
 /** Model of transaction search result */
 type SearchTransaction = {
     /** The ID of the transaction */
@@ -314,4 +351,15 @@ type SearchResults = {
 
 export default SearchResults;
 
-export type {ListItemType, ListItemDataType, SearchTransaction, SearchTransactionType, SearchTransactionAction, SearchPersonalDetails, SearchDataTypes, SearchReport, SearchReportAction};
+export type {
+    ListItemType,
+    ListItemDataType,
+    SearchTransaction,
+    SearchTransactionType,
+    SearchTransactionAction,
+    SearchPersonalDetails,
+    SearchDataTypes,
+    SearchReport,
+    SearchReportAction,
+    SearchPolicy,
+};

From d7593453cf1e6dc04feaf4e0d6560db068ef567e Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 24 Oct 2024 17:44:40 -0600
Subject: [PATCH 10/49] update searchpolicy type

---
 src/libs/SearchUIUtils.ts       | 2 +-
 src/libs/actions/IOU.ts         | 8 ++++----
 src/types/onyx/SearchResults.ts | 3 ++-
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 97c662da0521..d69b040dcc9c 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -263,7 +263,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     const transaction = isTransactionEntry(key) ? data[key] : null;
     const report = transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key];
     const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report?.chatReportID}`] ?? {};
-    const policy = data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.policyID}`] ?? {};
+    const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${transaction?.policyID}`] ?? {};
     const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
     const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
     const allTransactions = isTransactionEntry(key) ? transaction : Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION)) ?? {};
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 08fccd65a426..2a87e13ca3ba 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -66,7 +66,7 @@ import type {ErrorFields, Errors} from '@src/types/onyx/OnyxCommon';
 import type {PaymentMethodType} from '@src/types/onyx/OriginalMessage';
 import type ReportAction from '@src/types/onyx/ReportAction';
 import type {OnyxData} from '@src/types/onyx/Request';
-import type {SearchReport, SearchTransaction} from '@src/types/onyx/SearchResults';
+import type {SearchPolicy, SearchReport, SearchTransaction} from '@src/types/onyx/SearchResults';
 import type {Comment, Receipt, ReceiptSource, Routes, SplitShares, TransactionChanges, WaypointCollection} from '@src/types/onyx/Transaction';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import * as CachedPDFPaths from './CachedPDFPaths';
@@ -7016,7 +7016,7 @@ function sendMoneyWithWallet(report: OnyxEntry<OnyxTypes.Report>, amount: number
     Report.notifyNewAction(params.chatReportID, managerID);
 }
 
-function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport, policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>) {
+function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport, policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy) {
     // Only expense reports can be approved
     const isPaidGroupPolicy = policy && PolicyUtils.isPaidGroupPolicy(policy);
     if (!isPaidGroupPolicy) {
@@ -7043,7 +7043,7 @@ function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> |
 function canIOUBePaid(
     iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
     chatReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
-    policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>,
+    policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
     transactions?: OnyxTypes.Transaction[],
     onlyShowPayElsewhere = false,
 ) {
@@ -7106,7 +7106,7 @@ function canIOUBePaid(
 
 function canReportBeSubmitted(
     moneyRequestReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
-    policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy>,
+    policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
     currentUserAccountID: number,
     transaction?: OnyxTypes.Transaction,
 ) {
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index dda45454af00..bd1277805932 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -343,7 +343,8 @@ type SearchResults = {
     data: PrefixedRecord<typeof ONYXKEYS.COLLECTION.TRANSACTION, SearchTransaction> &
         Record<typeof ONYXKEYS.PERSONAL_DETAILS_LIST, Record<string, SearchPersonalDetails>> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_ACTIONS, Record<string, SearchReportAction>> &
-        PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT, SearchReport>;
+        PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT, SearchReport> &
+        PrefixedRecord<typeof ONYXKEYS.COLLECTION.POLICY, SearchPolicy>;
 
     /** Whether search data is being fetched from server */
     isLoading?: boolean;

From ad9081af3509c3c1acfd7dcc9f66f0e647437eda Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 25 Oct 2024 13:17:52 -0600
Subject: [PATCH 11/49] update button logic and styles

---
 src/components/SelectionList/Search/ActionCell.tsx | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/components/SelectionList/Search/ActionCell.tsx b/src/components/SelectionList/Search/ActionCell.tsx
index 275767ba6b68..d922f0a488d2 100644
--- a/src/components/SelectionList/Search/ActionCell.tsx
+++ b/src/components/SelectionList/Search/ActionCell.tsx
@@ -80,7 +80,6 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
         ) : null;
     }
 
-    // if (action === CONST.SEARCH.ACTION_TYPES.REVIEW) {
     return (
         <Button
             text={text}
@@ -88,9 +87,9 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
             small
             style={[styles.w100]}
             innerStyles={buttonInnerStyles}
+            success
         />
     );
-    // }
 }
 
 ActionCell.displayName = 'ActionCell';

From 21cf24d58af9a500ae2b993da4274d4f214f7dca Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 25 Oct 2024 15:00:48 -0600
Subject: [PATCH 12/49] use getActions in reports

---
 src/components/Search/index.tsx | 2 +-
 src/libs/SearchUIUtils.ts       | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/components/Search/index.tsx b/src/components/Search/index.tsx
index 5ff0882ea026..564621eadfa9 100644
--- a/src/components/Search/index.tsx
+++ b/src/components/Search/index.tsx
@@ -240,7 +240,7 @@ function Search({queryJSON, onSearchListScroll, contentContainerStyle}: SearchPr
                 });
             });
         }
-        setSelectedTransactions(newTransactionList, data);
+        // setSelectedTransactions(newTransactionList, data);
         // eslint-disable-next-line react-compiler/react-compiler, react-hooks/exhaustive-deps
     }, [data, setSelectedTransactions]);
 
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index d69b040dcc9c..32771715dffe 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -335,6 +335,7 @@ function getReportActionsSections(data: OnyxTypes.SearchResults['data']): Report
  * Do not use directly, use only via `getSections()` facade.
  */
 function getReportSections(data: OnyxTypes.SearchResults['data'], metadata: OnyxTypes.SearchResults['search']): ReportListItemType[] {
+    const currentUserAccountID = Report.getCurrentUserAccountID();
     const shouldShowMerchant = getShouldShowMerchant(data);
 
     const doesDataContainAPastYearTransaction = shouldShowYear(data);
@@ -366,6 +367,7 @@ function getReportSections(data: OnyxTypes.SearchResults['data'], metadata: Onyx
 
             const transaction = {
                 ...transactionItem,
+                action: getAction(data, key, currentUserAccountID),
                 from,
                 to,
                 formattedFrom,

From a1bcb5d798856c475233715a58fd117876c61802 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 25 Oct 2024 15:20:30 -0600
Subject: [PATCH 13/49] update isSettled

---
 src/libs/ReportUtils.ts   | 12 +++++++----
 src/libs/SearchUIUtils.ts | 43 +++++++++++++++++++--------------------
 2 files changed, 29 insertions(+), 26 deletions(-)

diff --git a/src/libs/ReportUtils.ts b/src/libs/ReportUtils.ts
index 6bc027fcc45d..891064f02eaa 100644
--- a/src/libs/ReportUtils.ts
+++ b/src/libs/ReportUtils.ts
@@ -57,6 +57,7 @@ import type {Message, ReportActions} from '@src/types/onyx/ReportAction';
 import type {Comment, TransactionChanges, WaypointCollection} from '@src/types/onyx/Transaction';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import type IconAsset from '@src/types/utils/IconAsset';
+import type {SearchReport} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
 import * as PolicyActions from './actions/Policy/Policy';
 import * as store from './actions/ReimbursementAccount/store';
@@ -950,12 +951,15 @@ function hasParticipantInArray(report: OnyxEntry<Report>, memberAccountIDs: numb
 /**
  * Whether the Money Request report is settled
  */
-function isSettled(reportID: string | undefined): boolean {
-    const allReports = ReportConnection.getAllReports();
-    if (!allReports || !reportID) {
+function isSettled(reportOrID: OnyxEntry<Report> | SearchReport | string | undefined): boolean {
+    if (!reportOrID) {
+        return false;
+    }
+    const report = typeof reportOrID === 'string' ? ReportConnection.getAllReports()?.[`${ONYXKEYS.COLLECTION.REPORT}${reportOrID}`] ?? null : reportOrID;
+    if (!report) {
         return false;
     }
-    const report = allReports[`${ONYXKEYS.COLLECTION.REPORT}${reportID}`] ?? null;
+
     if (isEmptyObject(report) || report.isWaitingOnBankAccount) {
         return false;
     }
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 32771715dffe..67b77e953524 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -268,32 +268,31 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
     const allTransactions = isTransactionEntry(key) ? transaction : Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION)) ?? {};
 
-    // TODO: update isSettled to take report object instead of connect to Onyx
-    if (ReportUtils.isSettled(report?.reportID)) {
+    if (ReportUtils.isSettled(report)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
     }
 
     // TODO: create function to determine if report is done
-    if (ReportUtils.isSettled(report?.reportID)) {
-        return CONST.SEARCH.ACTION_TYPES.DONE;
-    }
-
-    if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
-        return CONST.SEARCH.ACTION_TYPES.PAY;
-    }
-
-    if (IOU.canApproveIOU(report, policy)) {
-        return CONST.SEARCH.ACTION_TYPES.APPROVE;
-    }
-
-    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
-        return CONST.SEARCH.ACTION_TYPES.SUBMIT;
-    }
-
-    const hasViolations = isTransactionEntry(key) ? TransactionUtils.hasViolation(transaction.transactionID, violations) : ReportUtils.hasViolations(report.reportID, allViolations);
-    if (hasViolations) {
-        return CONST.SEARCH.ACTION_TYPES.REVIEW;
-    }
+    // if (ReportUtils.isSettled(report?.reportID)) {
+    //     return CONST.SEARCH.ACTION_TYPES.DONE;
+    // }
+
+    // if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
+    //     return CONST.SEARCH.ACTION_TYPES.PAY;
+    // }
+
+    // if (IOU.canApproveIOU(report, policy)) {
+    //     return CONST.SEARCH.ACTION_TYPES.APPROVE;
+    // }
+
+    // if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
+    //     return CONST.SEARCH.ACTION_TYPES.SUBMIT;
+    // }
+
+    // const hasViolations = isTransactionEntry(key) ? TransactionUtils.hasViolation(transaction.transactionID, violations) : ReportUtils.hasViolations(report.reportID, allViolations);
+    // if (hasViolations) {
+    //     return CONST.SEARCH.ACTION_TYPES.REVIEW;
+    // }
 
     return CONST.SEARCH.ACTION_TYPES.VIEW;
 }

From bd272b437b9bbdef5981d76aad8c821fd3abceed Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 25 Oct 2024 15:28:24 -0600
Subject: [PATCH 14/49] add done condition

---
 src/libs/ReportUtils.ts   | 2 +-
 src/libs/SearchUIUtils.ts | 7 +++----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/libs/ReportUtils.ts b/src/libs/ReportUtils.ts
index 891064f02eaa..6602a557a60a 100644
--- a/src/libs/ReportUtils.ts
+++ b/src/libs/ReportUtils.ts
@@ -1452,7 +1452,7 @@ function isArchivedRoomWithID(reportID?: string) {
 /**
  * Whether the provided report is a closed report
  */
-function isClosedReport(report: OnyxEntry<Report>): boolean {
+function isClosedReport(report: OnyxEntry<Report> | SearchReport): boolean {
     return report?.statusNum === CONST.REPORT.STATUS_NUM.CLOSED;
 }
 
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 67b77e953524..98bdae487757 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -272,10 +272,9 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.PAID;
     }
 
-    // TODO: create function to determine if report is done
-    // if (ReportUtils.isSettled(report?.reportID)) {
-    //     return CONST.SEARCH.ACTION_TYPES.DONE;
-    // }
+    if (ReportUtils.isClosedReport(report)) {
+        return CONST.SEARCH.ACTION_TYPES.DONE;
+    }
 
     // if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
     //     return CONST.SEARCH.ACTION_TYPES.PAY;

From f87db13728e7e167f4577f394c61f5ee2a1d39ba Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 25 Oct 2024 16:30:53 -0600
Subject: [PATCH 15/49] fix logic for all transactions

---
 src/libs/SearchUIUtils.ts       | 19 +++++++++++++------
 src/libs/actions/IOU.ts         |  2 +-
 src/types/onyx/SearchResults.ts |  3 +++
 3 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 98bdae487757..965fb2bc8df9 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -261,12 +261,18 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     const transaction = isTransactionEntry(key) ? data[key] : null;
-    const report = transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key];
+    const report = transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : (data[key] as SearchReport);
+
+    // We don't need to run the logic if this is not a transaction or iou/expense report
+    if (!ReportUtils.isMoneyRequestReport(report)) {
+        return CONST.SEARCH.ACTION_TYPES.VIEW;
+    }
+
     const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report?.chatReportID}`] ?? {};
-    const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${transaction?.policyID}`] ?? {};
+    const allTransactions = Object.values(data).filter((item) => item?.transactionID && item?.reportID === report.reportID);
+    const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`] ?? {};
     const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
     const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
-    const allTransactions = isTransactionEntry(key) ? transaction : Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION)) ?? {};
 
     if (ReportUtils.isSettled(report)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
@@ -276,9 +282,9 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.DONE;
     }
 
-    // if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
-    //     return CONST.SEARCH.ACTION_TYPES.PAY;
-    // }
+    if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
+        return CONST.SEARCH.ACTION_TYPES.PAY;
+    }
 
     // if (IOU.canApproveIOU(report, policy)) {
     //     return CONST.SEARCH.ACTION_TYPES.APPROVE;
@@ -348,6 +354,7 @@ function getReportSections(data: OnyxTypes.SearchResults['data'], metadata: Onyx
 
             reportIDToTransactions[reportKey] = {
                 ...reportItem,
+                action: getAction(data, key, currentUserAccountID),
                 keyForList: reportItem.reportID,
                 from: data.personalDetailsList?.[reportItem.accountID ?? -1],
                 to: reportItem.managerID ? data.personalDetailsList?.[reportItem.managerID] : emptyPersonalDetails,
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 2a87e13ca3ba..7222b40813d8 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7044,7 +7044,7 @@ function canIOUBePaid(
     iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
     chatReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
     policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
-    transactions?: OnyxTypes.Transaction[],
+    transactions?: OnyxTypes.Transaction[] | SearchTransaction[],
     onlyShowPayElsewhere = false,
 ) {
     const isPolicyExpenseChat = ReportUtils.isPolicyExpenseChat(chatReport);
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index bd1277805932..f06224190db6 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -81,6 +81,9 @@ type SearchReport = {
     /** The ID of the report */
     reportID?: string;
 
+    /** ID of the chat report */
+    chatReportID?: string;
+
     /** The name of the report */
     reportName?: string;
 

From 1de744c5e1af6ae002deb87404ed78dc5dd7ba5b Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 25 Oct 2024 16:37:45 -0600
Subject: [PATCH 16/49] enable approve logic

---
 src/libs/SearchUIUtils.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 965fb2bc8df9..797558b2c69c 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -286,9 +286,9 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.PAY;
     }
 
-    // if (IOU.canApproveIOU(report, policy)) {
-    //     return CONST.SEARCH.ACTION_TYPES.APPROVE;
-    // }
+    if (IOU.canApproveIOU(report, policy)) {
+        return CONST.SEARCH.ACTION_TYPES.APPROVE;
+    }
 
     // if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
     //     return CONST.SEARCH.ACTION_TYPES.SUBMIT;

From 53421fc65108bb126df01b925aab04d31d629853 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 25 Oct 2024 16:43:42 -0600
Subject: [PATCH 17/49] refactor code

---
 src/libs/SearchUIUtils.ts | 7 ++++---
 src/libs/actions/IOU.ts   | 2 +-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 797558b2c69c..1107e2c79d7a 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -290,9 +290,10 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
 
-    // if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
-    //     return CONST.SEARCH.ACTION_TYPES.SUBMIT;
-    // }
+    // TODO: update allHavePendingRTERViolation to take in array of transactions
+    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
+        return CONST.SEARCH.ACTION_TYPES.SUBMIT;
+    }
 
     // const hasViolations = isTransactionEntry(key) ? TransactionUtils.hasViolation(transaction.transactionID, violations) : ReportUtils.hasViolations(report.reportID, allViolations);
     // if (hasViolations) {
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 7222b40813d8..6e67832774b6 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7108,7 +7108,7 @@ function canReportBeSubmitted(
     moneyRequestReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
     policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
     currentUserAccountID: number,
-    transaction?: OnyxTypes.Transaction,
+    transaction?: OnyxTypes.Transaction | SearchTransaction,
 ) {
     const isDraft = ReportUtils.isOpenExpenseReport(moneyRequestReport);
     const {reimbursableSpend} = ReportUtils.getMoneyRequestSpendBreakdown(moneyRequestReport);

From 82a7f0c29b3fcd392a58623e895fda33653e966b Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Mon, 28 Oct 2024 17:30:36 -0600
Subject: [PATCH 18/49] fix some ts errors

---
 src/libs/ReportUtils.ts   | 6 +++---
 src/libs/SearchUIUtils.ts | 8 +++++---
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/libs/ReportUtils.ts b/src/libs/ReportUtils.ts
index f087c14c5c5c..dc4d119169e3 100644
--- a/src/libs/ReportUtils.ts
+++ b/src/libs/ReportUtils.ts
@@ -835,14 +835,14 @@ function isInvoiceReport(report: OnyxInputOrEntry<Report>): boolean {
 /**
  * Checks if a report is an Expense report.
  */
-function isExpenseReport(report: OnyxInputOrEntry<Report>): boolean {
+function isExpenseReport(report: OnyxInputOrEntry<Report> | SearchReport): boolean {
     return report?.type === CONST.REPORT.TYPE.EXPENSE;
 }
 
 /**
  * Checks if a report is an IOU report using report or reportID
  */
-function isIOUReport(reportOrID: OnyxInputOrEntry<Report> | string): boolean {
+function isIOUReport(reportOrID: OnyxInputOrEntry<Report> | SearchReport| string): boolean {
     const report = typeof reportOrID === 'string' ? ReportConnection.getAllReports()?.[`${ONYXKEYS.COLLECTION.REPORT}${reportOrID}`] ?? null : reportOrID;
     return report?.type === CONST.REPORT.TYPE.IOU;
 }
@@ -1639,7 +1639,7 @@ function isMoneyRequest(reportOrID: OnyxEntry<Report> | string): boolean {
 /**
  * Checks if a report is an IOU or expense report.
  */
-function isMoneyRequestReport(reportOrID: OnyxInputOrEntry<Report> | string): boolean {
+function isMoneyRequestReport(reportOrID: OnyxInputOrEntry<Report> | SearchReport | string): boolean {
     const report = typeof reportOrID === 'string' ? ReportConnection.getAllReports()?.[`${ONYXKEYS.COLLECTION.REPORT}${reportOrID}`] ?? null : reportOrID;
     return isIOUReport(report) || isExpenseReport(report);
 }
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 1107e2c79d7a..cd3420d130bd 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -261,7 +261,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     const transaction = isTransactionEntry(key) ? data[key] : null;
-    const report = transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : (data[key] as SearchReport);
+    const report = (transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key]) as SearchReport;
 
     // We don't need to run the logic if this is not a transaction or iou/expense report
     if (!ReportUtils.isMoneyRequestReport(report)) {
@@ -269,7 +269,8 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report?.chatReportID}`] ?? {};
-    const allTransactions = Object.values(data).filter((item) => item?.transactionID && item?.reportID === report.reportID);
+    const allReportTransactions = isReportEntry(key) ? Object.entries(data).filter(([itemKey, value]) => isTransactionEntry(itemKey) && (value as SearchTransaction)?.reportID === report.reportID).map(item => item[1]) : [];
+    console.log('over here', allReportTransactions)
     const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`] ?? {};
     const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
     const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
@@ -282,7 +283,8 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.DONE;
     }
 
-    if (IOU.canIOUBePaid(report, chatReport, policy, allTransactions, false)) {
+    // TODO: Fix TS error
+    if (IOU.canIOUBePaid(report, chatReport, policy, allReportTransactions, false)) {
         return CONST.SEARCH.ACTION_TYPES.PAY;
     }
 

From 4445a3a9627bc18123a5179ad3f0620bb1ead095 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Mon, 28 Oct 2024 18:07:57 -0600
Subject: [PATCH 19/49] fix more ts errors

---
 src/libs/PolicyUtils.ts         |  5 +++--
 src/libs/ReportUtils.ts         |  8 ++++----
 src/libs/SearchUIUtils.ts       | 18 ++++++++++++++----
 src/types/onyx/Report.ts        |  2 +-
 src/types/onyx/SearchResults.ts | 14 ++++++++++----
 5 files changed, 32 insertions(+), 15 deletions(-)

diff --git a/src/libs/PolicyUtils.ts b/src/libs/PolicyUtils.ts
index 36e5ccef3308..33e3b0c0d459 100644
--- a/src/libs/PolicyUtils.ts
+++ b/src/libs/PolicyUtils.ts
@@ -35,6 +35,7 @@ import * as Localize from './Localize';
 import Navigation from './Navigation/Navigation';
 import * as NetworkStore from './Network/NetworkStore';
 import {getAccountIDsByLogins, getLoginsByAccountIDs, getPersonalDetailByEmail} from './PersonalDetailsUtils';
+import { SearchPolicy } from '@src/types/onyx/SearchResults';
 
 type MemberEmailsToAccountIDs = Record<string, number>;
 
@@ -369,7 +370,7 @@ function isPendingDeletePolicy(policy: OnyxEntry<Policy>): boolean {
     return policy?.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE;
 }
 
-function isPaidGroupPolicy(policy: OnyxEntry<Policy>): boolean {
+function isPaidGroupPolicy(policy: OnyxEntry<Policy> | SearchPolicy): boolean {
     return policy?.type === CONST.POLICY.TYPE.TEAM || policy?.type === CONST.POLICY.TYPE.CORPORATE;
 }
 
@@ -424,7 +425,7 @@ function getCorrectedAutoReportingFrequency(policy: OnyxInputOrEntry<Policy>): V
 /**
  * Checks if policy's approval mode is "optional", a.k.a. "Submit & Close"
  */
-function isSubmitAndClose(policy: OnyxInputOrEntry<Policy>): boolean {
+function isSubmitAndClose(policy: OnyxInputOrEntry<Policy> | SearchPolicy): boolean {
     return policy?.approvalMode === CONST.POLICY.APPROVAL_MODE.OPTIONAL;
 }
 
diff --git a/src/libs/ReportUtils.ts b/src/libs/ReportUtils.ts
index dc4d119169e3..4233572a3051 100644
--- a/src/libs/ReportUtils.ts
+++ b/src/libs/ReportUtils.ts
@@ -57,7 +57,7 @@ import type {Message, ReportActions} from '@src/types/onyx/ReportAction';
 import type {Comment, TransactionChanges, WaypointCollection} from '@src/types/onyx/Transaction';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import type IconAsset from '@src/types/utils/IconAsset';
-import type {SearchReport} from '@src/types/onyx/SearchResults';
+import type {SearchPolicy, SearchReport} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
 import * as PolicyActions from './actions/Policy/Policy';
 import * as store from './actions/ReimbursementAccount/store';
@@ -828,7 +828,7 @@ function isChatReport(report: OnyxEntry<Report>): boolean {
     return report?.type === CONST.REPORT.TYPE.CHAT;
 }
 
-function isInvoiceReport(report: OnyxInputOrEntry<Report>): boolean {
+function isInvoiceReport(report: OnyxInputOrEntry<Report> | SearchReport): boolean {
     return report?.type === CONST.REPORT.TYPE.INVOICE;
 }
 
@@ -1436,7 +1436,7 @@ function isClosedExpenseReportWithNoExpenses(report: OnyxEntry<Report>): boolean
  * Whether the provided report is an archived room
  */
 // eslint-disable-next-line @typescript-eslint/no-unused-vars
-function isArchivedRoom(report: OnyxInputOrEntry<Report>, reportNameValuePairs?: OnyxInputOrEntry<ReportNameValuePairs>): boolean {
+function isArchivedRoom(report: OnyxInputOrEntry<Report> | SearchReport, reportNameValuePairs?: OnyxInputOrEntry<ReportNameValuePairs>): boolean {
     return !!report?.private_isArchived;
 }
 
@@ -7870,7 +7870,7 @@ function getOptimisticDataForParentReportAction(reportID: string, lastVisibleAct
     });
 }
 
-function canBeAutoReimbursed(report: OnyxInputOrEntry<Report>, policy: OnyxInputOrEntry<Policy>): boolean {
+function canBeAutoReimbursed(report: OnyxInputOrEntry<Report>, policy: OnyxInputOrEntry<Policy> | SearchPolicy): boolean {
     if (isEmptyObject(policy)) {
         return false;
     }
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index cd3420d130bd..e159340b66d2 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -12,6 +12,7 @@ import ROUTES from '@src/ROUTES';
 import type * as OnyxTypes from '@src/types/onyx';
 import type SearchResults from '@src/types/onyx/SearchResults';
 import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction, SearchPolicy} from '@src/types/onyx/SearchResults';
+import type {TransactionViolation} from '@src/types/onyx';
 import * as IOU from './actions/IOU';
 import * as Report from './actions/Report';
 import * as CurrencyUtils from './CurrencyUtils';
@@ -260,7 +261,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.VIEW;
     }
 
-    const transaction = isTransactionEntry(key) ? data[key] : null;
+    const transaction = isTransactionEntry(key) ? data[key] : undefined;
     const report = (transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key]) as SearchReport;
 
     // We don't need to run the logic if this is not a transaction or iou/expense report
@@ -269,8 +270,13 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report?.chatReportID}`] ?? {};
-    const allReportTransactions = isReportEntry(key) ? Object.entries(data).filter(([itemKey, value]) => isTransactionEntry(itemKey) && (value as SearchTransaction)?.reportID === report.reportID).map(item => item[1]) : [];
-    console.log('over here', allReportTransactions)
+    const allReportTransactions = (
+        isReportEntry(key)
+            ? Object.entries(data)
+                  .filter(([itemKey, value]) => isTransactionEntry(itemKey) && (value as SearchTransaction)?.reportID === report.reportID)
+                  .map((item) => item[1])
+            : []
+    ) as SearchTransaction[];
     const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`] ?? {};
     const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
     const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
@@ -283,11 +289,15 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.DONE;
     }
 
-    // TODO: Fix TS error
+    // TODO: pass reportNameValuePairs here so we don't get it in ReportUtils.isArchivedRoom and canApproveIOU
+    // TODO: pass receiver policy so that we don't get it from Onyx in PolicyUtils.getPolicy
+    // TODO: pass policy to ReportUtils.isPayer and isPaidGroupPolicy
+    // TODO: in Auth, send invoiceReceiver as accountID if type is individual
     if (IOU.canIOUBePaid(report, chatReport, policy, allReportTransactions, false)) {
         return CONST.SEARCH.ACTION_TYPES.PAY;
     }
 
+    // TODO: pass reportNameValuePairs to ReportUtils.getReportNameValuePairs
     if (IOU.canApproveIOU(report, policy)) {
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
diff --git a/src/types/onyx/Report.ts b/src/types/onyx/Report.ts
index 43c82cfdc227..c4b9ebe97b9d 100644
--- a/src/types/onyx/Report.ts
+++ b/src/types/onyx/Report.ts
@@ -311,4 +311,4 @@ type ReportCollectionDataSet = CollectionDataSet<typeof ONYXKEYS.COLLECTION.REPO
 
 export default Report;
 
-export type {NotificationPreference, RoomVisibility, WriteCapability, Note, ReportCollectionDataSet, PendingChatMember, Participant, Participants};
+export type {NotificationPreference, RoomVisibility, WriteCapability, Note, ReportCollectionDataSet, PendingChatMember, Participant, Participants, InvoiceReceiver};
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index f06224190db6..c04c597fef0a 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -6,7 +6,9 @@ import type TransactionListItem from '@components/SelectionList/Search/Transacti
 import type {ReportActionListItemType, ReportListItemType, TransactionListItemType} from '@components/SelectionList/types';
 import type CONST from '@src/CONST';
 import type ONYXKEYS from '@src/ONYXKEYS';
+import type {InvoiceReceiver} from './Report';
 import type ReportActionName from './ReportActionName';
+import type {TransactionViolation} from './TransactionViolation';
 
 /** Types of search data */
 type SearchDataTypes = ValueOf<typeof CONST.SEARCH.DATA_TYPES>;
@@ -79,7 +81,7 @@ type SearchTransactionAction = ValueOf<typeof CONST.SEARCH.ACTION_TYPES>;
 /** Model of report search result */
 type SearchReport = {
     /** The ID of the report */
-    reportID?: string;
+    reportID: string;
 
     /** ID of the chat report */
     chatReportID?: string;
@@ -115,7 +117,7 @@ type SearchReport = {
     chatType?: ValueOf<typeof CONST.REPORT.CHAT_TYPE>;
 
     /** Invoice room receiver data */
-    invoiceReceiver?: Record<string, string>;
+    invoiceReceiver?: InvoiceReceiver;
 
     /** Whether the report has a single transaction */
     isOneTransactionReport?: boolean;
@@ -213,7 +215,10 @@ type SearchPolicy = {
     employeeList?: Record<string, Record<string, string>>;
 
     /** Detailed settings for the autoReimbursement */
-    autoReimbursement?: Record<string, string>;
+    autoReimbursement?: {
+        /** The auto reimbursement limit */
+        limit: number;
+    };
 };
 
 /** Model of transaction search result */
@@ -347,7 +352,8 @@ type SearchResults = {
         Record<typeof ONYXKEYS.PERSONAL_DETAILS_LIST, Record<string, SearchPersonalDetails>> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_ACTIONS, Record<string, SearchReportAction>> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT, SearchReport> &
-        PrefixedRecord<typeof ONYXKEYS.COLLECTION.POLICY, SearchPolicy>;
+        PrefixedRecord<typeof ONYXKEYS.COLLECTION.POLICY, SearchPolicy> &
+        PrefixedRecord<typeof ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS, TransactionViolation>;
 
     /** Whether search data is being fetched from server */
     isLoading?: boolean;

From f9055ad56e369a118aa08fa73f86981335a2ac82 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Mon, 28 Oct 2024 18:11:27 -0600
Subject: [PATCH 20/49] fix more ts errors

---
 src/libs/PolicyUtils.ts            | 6 +++---
 src/libs/SearchUIUtils.ts          | 1 +
 src/libs/TransactionUtils/index.ts | 3 ++-
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/libs/PolicyUtils.ts b/src/libs/PolicyUtils.ts
index 33e3b0c0d459..f7ce47747247 100644
--- a/src/libs/PolicyUtils.ts
+++ b/src/libs/PolicyUtils.ts
@@ -176,7 +176,7 @@ function getPolicyBrickRoadIndicatorStatus(policy: OnyxEntry<Policy>, isConnecti
     return undefined;
 }
 
-function getPolicyRole(policy: OnyxInputOrEntry<Policy>, currentUserLogin: string | undefined) {
+function getPolicyRole(policy: OnyxInputOrEntry<Policy> | SearchPolicy, currentUserLogin: string | undefined) {
     return policy?.role ?? policy?.employeeList?.[currentUserLogin ?? '-1']?.role;
 }
 
@@ -209,7 +209,7 @@ const isUserPolicyAdmin = (policy: OnyxInputOrEntry<Policy>, login?: string) =>
 /**
  * Checks if the current user is an admin of the policy.
  */
-const isPolicyAdmin = (policy: OnyxInputOrEntry<Policy>, currentUserLogin?: string): boolean => getPolicyRole(policy, currentUserLogin) === CONST.POLICY.ROLE.ADMIN;
+const isPolicyAdmin = (policy: OnyxInputOrEntry<Policy> | SearchPolicy, currentUserLogin?: string): boolean => getPolicyRole(policy, currentUserLogin) === CONST.POLICY.ROLE.ADMIN;
 
 /**
  * Checks if the current user is of the role "user" on the policy.
@@ -395,7 +395,7 @@ function isTaxTrackingEnabled(isPolicyExpenseChat: boolean, policy: OnyxEntry<Po
  * Checks if policy's scheduled submit / auto reporting frequency is "instant".
  * Note: Free policies have "instant" submit always enabled.
  */
-function isInstantSubmitEnabled(policy: OnyxInputOrEntry<Policy>): boolean {
+function isInstantSubmitEnabled(policy: OnyxInputOrEntry<Policy> | SearchPolicy): boolean {
     return policy?.autoReporting === true && policy?.autoReportingFrequency === CONST.POLICY.AUTO_REPORTING_FREQUENCIES.INSTANT;
 }
 
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index e159340b66d2..af2fa767b061 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -303,6 +303,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     // TODO: update allHavePendingRTERViolation to take in array of transactions
+    // TODO: update hasBrokenConnectionViolation to take in violations instead of connecting directly to Onyx
     if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
         return CONST.SEARCH.ACTION_TYPES.SUBMIT;
     }
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index bf5073aba27c..3f048e461d36 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -28,6 +28,7 @@ import type {Comment, Receipt, TransactionChanges, TransactionPendingFieldsKey,
 import type DeepValueOf from '@src/types/utils/DeepValueOf';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import getDistanceInMeters from './getDistanceInMeters';
+import { SearchPolicy, SearchReport } from '@src/types/onyx/SearchResults';
 
 let allTransactions: OnyxCollection<Transaction> = {};
 Onyx.connect({
@@ -659,7 +660,7 @@ function hasBrokenConnectionViolation(transactionID: string): boolean {
 /**
  * Check if user should see broken connection violation warning.
  */
-function shouldShowBrokenConnectionViolation(transactionID: string, report: OnyxEntry<Report>, policy: OnyxEntry<Policy>): boolean {
+function shouldShowBrokenConnectionViolation(transactionID: string, report: OnyxEntry<Report> | SearchReport, policy: OnyxEntry<Policy> | SearchPolicy): boolean {
     return (
         hasBrokenConnectionViolation(transactionID) &&
         (!PolicyUtils.isPolicyAdmin(policy) || ReportUtils.isOpenExpenseReport(report) || (ReportUtils.isProcessingReport(report) && PolicyUtils.isInstantSubmitEnabled(policy)))

From b8568f4e04cf47f77f5ca5a0bc80e0a86f3387c9 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Mon, 28 Oct 2024 18:17:21 -0600
Subject: [PATCH 21/49] fix lint

---
 src/libs/PolicyUtils.ts            |  2 +-
 src/libs/ReportUtils.ts            |  4 ++--
 src/libs/SearchUIUtils.ts          | 13 +++++++++++--
 src/libs/TransactionUtils/index.ts |  2 +-
 4 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/src/libs/PolicyUtils.ts b/src/libs/PolicyUtils.ts
index f7ce47747247..0174f39c2657 100644
--- a/src/libs/PolicyUtils.ts
+++ b/src/libs/PolicyUtils.ts
@@ -29,13 +29,13 @@ import type {
     Tenant,
 } from '@src/types/onyx/Policy';
 import type PolicyEmployee from '@src/types/onyx/PolicyEmployee';
+import {SearchPolicy} from '@src/types/onyx/SearchResults';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import {hasSynchronizationErrorMessage} from './actions/connections';
 import * as Localize from './Localize';
 import Navigation from './Navigation/Navigation';
 import * as NetworkStore from './Network/NetworkStore';
 import {getAccountIDsByLogins, getLoginsByAccountIDs, getPersonalDetailByEmail} from './PersonalDetailsUtils';
-import { SearchPolicy } from '@src/types/onyx/SearchResults';
 
 type MemberEmailsToAccountIDs = Record<string, number>;
 
diff --git a/src/libs/ReportUtils.ts b/src/libs/ReportUtils.ts
index 4233572a3051..b1e28990dd8d 100644
--- a/src/libs/ReportUtils.ts
+++ b/src/libs/ReportUtils.ts
@@ -54,10 +54,10 @@ import type {Status} from '@src/types/onyx/PersonalDetails';
 import type {ConnectionName} from '@src/types/onyx/Policy';
 import type {NotificationPreference, Participants, PendingChatMember, Participant as ReportParticipant} from '@src/types/onyx/Report';
 import type {Message, ReportActions} from '@src/types/onyx/ReportAction';
+import type {SearchPolicy, SearchReport} from '@src/types/onyx/SearchResults';
 import type {Comment, TransactionChanges, WaypointCollection} from '@src/types/onyx/Transaction';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import type IconAsset from '@src/types/utils/IconAsset';
-import type {SearchPolicy, SearchReport} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
 import * as PolicyActions from './actions/Policy/Policy';
 import * as store from './actions/ReimbursementAccount/store';
@@ -842,7 +842,7 @@ function isExpenseReport(report: OnyxInputOrEntry<Report> | SearchReport): boole
 /**
  * Checks if a report is an IOU report using report or reportID
  */
-function isIOUReport(reportOrID: OnyxInputOrEntry<Report> | SearchReport| string): boolean {
+function isIOUReport(reportOrID: OnyxInputOrEntry<Report> | SearchReport | string): boolean {
     const report = typeof reportOrID === 'string' ? ReportConnection.getAllReports()?.[`${ONYXKEYS.COLLECTION.REPORT}${reportOrID}`] ?? null : reportOrID;
     return report?.type === CONST.REPORT.TYPE.IOU;
 }
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index af2fa767b061..806ed12c1ee2 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -10,9 +10,18 @@ import type {TranslationPaths} from '@src/languages/types';
 import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
 import type * as OnyxTypes from '@src/types/onyx';
-import type SearchResults from '@src/types/onyx/SearchResults';
-import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction, SearchPolicy} from '@src/types/onyx/SearchResults';
 import type {TransactionViolation} from '@src/types/onyx';
+import type SearchResults from '@src/types/onyx/SearchResults';
+import type {
+    ListItemDataType,
+    ListItemType,
+    SearchDataTypes,
+    SearchPersonalDetails,
+    SearchPolicy,
+    SearchReport,
+    SearchTransaction,
+    SearchTransactionAction,
+} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
 import * as Report from './actions/Report';
 import * as CurrencyUtils from './CurrencyUtils';
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index 3f048e461d36..5a96cf3c34c8 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -24,11 +24,11 @@ import type {IOUType} from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import type {Beta, OnyxInputOrEntry, Policy, RecentWaypoint, Report, ReviewDuplicates, TaxRate, TaxRates, Transaction, TransactionViolation, TransactionViolations} from '@src/types/onyx';
 import type {Attendee} from '@src/types/onyx/IOU';
+import {SearchPolicy, SearchReport} from '@src/types/onyx/SearchResults';
 import type {Comment, Receipt, TransactionChanges, TransactionPendingFieldsKey, Waypoint, WaypointCollection} from '@src/types/onyx/Transaction';
 import type DeepValueOf from '@src/types/utils/DeepValueOf';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import getDistanceInMeters from './getDistanceInMeters';
-import { SearchPolicy, SearchReport } from '@src/types/onyx/SearchResults';
 
 let allTransactions: OnyxCollection<Transaction> = {};
 Onyx.connect({

From 97ee8fc1f4102e0bb477d853bda0e038492a710e Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Tue, 29 Oct 2024 14:28:28 -0600
Subject: [PATCH 22/49] finish requirements for pay action

---
 src/libs/ReportUtils.ts         |  6 +++---
 src/libs/SearchUIUtils.ts       | 11 ++++++-----
 src/libs/actions/IOU.ts         |  9 ++++++---
 src/types/onyx/SearchResults.ts |  2 ++
 4 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/src/libs/ReportUtils.ts b/src/libs/ReportUtils.ts
index b1e28990dd8d..f9accc482065 100644
--- a/src/libs/ReportUtils.ts
+++ b/src/libs/ReportUtils.ts
@@ -951,7 +951,7 @@ function hasParticipantInArray(report: OnyxEntry<Report>, memberAccountIDs: numb
 /**
  * Whether the Money Request report is settled
  */
-function isSettled(reportOrID: OnyxEntry<Report> | SearchReport | string | undefined): boolean {
+function isSettled(reportOrID: OnyxInputOrEntry<Report> | SearchReport | string | undefined): boolean {
     if (!reportOrID) {
         return false;
     }
@@ -1706,9 +1706,9 @@ function isOneOnOneChat(report: OnyxEntry<Report>): boolean {
  * Checks if the current user is a payer of the expense
  */
 
-function isPayer(session: OnyxEntry<Session>, iouReport: OnyxEntry<Report>, onlyShowPayElsewhere = false) {
+function isPayer(session: OnyxEntry<Session>, iouReport: OnyxEntry<Report>, onlyShowPayElsewhere = false, reportPolicy?: OnyxInputOrEntry<Policy> | SearchPolicy) {
     const isApproved = isReportApproved(iouReport);
-    const policy = allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${iouReport?.policyID}`] ?? null;
+    const policy = reportPolicy ?? allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${iouReport?.policyID}`] ?? null;
     const policyType = policy?.type;
     const isAdmin = policyType !== CONST.POLICY.TYPE.PERSONAL && policy?.role === CONST.POLICY.ROLE.ADMIN;
     const isManager = iouReport?.managerID === session?.accountID;
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 806ed12c1ee2..81567efeda67 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -279,6 +279,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report?.chatReportID}`] ?? {};
+    const chatReportRNVP = data[`${ONYXKEYS.COLLECTION.REPORT_NAME_VALUE_PAIRS}${report?.chatReportID}`] ?? undefined;
     const allReportTransactions = (
         isReportEntry(key)
             ? Object.entries(data)
@@ -287,6 +288,10 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
             : []
     ) as SearchTransaction[];
     const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`] ?? {};
+    const invoiceReceiverPolicy =
+        ReportUtils.isInvoiceReport(report) && report?.invoiceReceiver?.type === CONST.REPORT.INVOICE_RECEIVER_TYPE.BUSINESS
+            ? data[`${ONYXKEYS.COLLECTION.POLICY}${report?.invoiceReceiver?.policyID}`]
+            : undefined;
     const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
     const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
 
@@ -298,11 +303,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.DONE;
     }
 
-    // TODO: pass reportNameValuePairs here so we don't get it in ReportUtils.isArchivedRoom and canApproveIOU
-    // TODO: pass receiver policy so that we don't get it from Onyx in PolicyUtils.getPolicy
-    // TODO: pass policy to ReportUtils.isPayer and isPaidGroupPolicy
-    // TODO: in Auth, send invoiceReceiver as accountID if type is individual
-    if (IOU.canIOUBePaid(report, chatReport, policy, allReportTransactions, false)) {
+    if (IOU.canIOUBePaid(report, chatReport, policy, allReportTransactions, false, chatReportRNVP, invoiceReceiverPolicy)) {
         return CONST.SEARCH.ACTION_TYPES.PAY;
     }
 
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 6e67832774b6..b349baf113bc 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7046,11 +7046,13 @@ function canIOUBePaid(
     policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
     transactions?: OnyxTypes.Transaction[] | SearchTransaction[],
     onlyShowPayElsewhere = false,
+    chatReportRNVP?: OnyxTypes.ReportNameValuePairs,
+    invoiceReceiverPolicy?: SearchPolicy,
 ) {
     const isPolicyExpenseChat = ReportUtils.isPolicyExpenseChat(chatReport);
-    const reportNameValuePairs = ReportUtils.getReportNameValuePairs(chatReport?.reportID);
+    const reportNameValuePairs = chatReportRNVP ?? ReportUtils.getReportNameValuePairs(chatReport?.reportID);
     const isChatReportArchived = ReportUtils.isArchivedRoom(chatReport, reportNameValuePairs);
-    const iouSettled = ReportUtils.isSettled(iouReport?.reportID);
+    const iouSettled = ReportUtils.isSettled(iouReport);
 
     if (isEmptyObject(iouReport)) {
         return false;
@@ -7072,7 +7074,7 @@ function canIOUBePaid(
         if (chatReport?.invoiceReceiver?.type === CONST.REPORT.INVOICE_RECEIVER_TYPE.INDIVIDUAL) {
             return chatReport?.invoiceReceiver?.accountID === userAccountID;
         }
-        return PolicyUtils.getPolicy(chatReport?.invoiceReceiver?.policyID)?.role === CONST.POLICY.ROLE.ADMIN;
+        return (invoiceReceiverPolicy ?? PolicyUtils.getPolicy(chatReport?.invoiceReceiver?.policyID))?.role === CONST.POLICY.ROLE.ADMIN;
     }
 
     const isPayer = ReportUtils.isPayer(
@@ -7082,6 +7084,7 @@ function canIOUBePaid(
         },
         iouReport,
         onlyShowPayElsewhere,
+        policy,
     );
 
     const isOpenExpenseReport = isPolicyExpenseChat && ReportUtils.isOpenExpenseReport(iouReport);
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index c04c597fef0a..117eaf79c598 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -9,6 +9,7 @@ import type ONYXKEYS from '@src/ONYXKEYS';
 import type {InvoiceReceiver} from './Report';
 import type ReportActionName from './ReportActionName';
 import type {TransactionViolation} from './TransactionViolation';
+import ReportNameValuePairs from './ReportNameValuePairs';
 
 /** Types of search data */
 type SearchDataTypes = ValueOf<typeof CONST.SEARCH.DATA_TYPES>;
@@ -353,6 +354,7 @@ type SearchResults = {
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_ACTIONS, Record<string, SearchReportAction>> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT, SearchReport> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.POLICY, SearchPolicy> &
+        PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_NAME_VALUE_PAIRS, ReportNameValuePairs> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS, TransactionViolation>;
 
     /** Whether search data is being fetched from server */

From 8be05076b93d863ab1d76be1a248357b15ebe31d Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Tue, 29 Oct 2024 14:30:30 -0600
Subject: [PATCH 23/49] finish approve

---
 src/libs/SearchUIUtils.ts | 1 -
 src/libs/actions/IOU.ts   | 4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 81567efeda67..34f68466fe6b 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -307,7 +307,6 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.PAY;
     }
 
-    // TODO: pass reportNameValuePairs to ReportUtils.getReportNameValuePairs
     if (IOU.canApproveIOU(report, policy)) {
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index b349baf113bc..b7087c91a0db 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7016,7 +7016,7 @@ function sendMoneyWithWallet(report: OnyxEntry<OnyxTypes.Report>, amount: number
     Report.notifyNewAction(params.chatReportID, managerID);
 }
 
-function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport, policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy) {
+function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport, policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy, chatReportRNVP?: OnyxTypes.ReportNameValuePairs) {
     // Only expense reports can be approved
     const isPaidGroupPolicy = policy && PolicyUtils.isPaidGroupPolicy(policy);
     if (!isPaidGroupPolicy) {
@@ -7033,7 +7033,7 @@ function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> |
     const isOpenExpenseReport = ReportUtils.isOpenExpenseReport(iouReport);
     const isApproved = ReportUtils.isReportApproved(iouReport);
     const iouSettled = ReportUtils.isSettled(iouReport?.reportID);
-    const reportNameValuePairs = ReportUtils.getReportNameValuePairs(iouReport?.reportID);
+    const reportNameValuePairs = chatReportRNVP ?? ReportUtils.getReportNameValuePairs(iouReport?.reportID);
     const isArchivedReport = ReportUtils.isArchivedRoom(iouReport, reportNameValuePairs);
     const unheldTotalIsZero = iouReport && iouReport.unheldTotal === 0;
 

From faa0ba482255a31120bc499a9179266c2c46d239 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Tue, 29 Oct 2024 14:58:40 -0600
Subject: [PATCH 24/49] pass violations to submit

---
 src/libs/SearchUIUtils.ts          | 21 ++++++---------------
 src/libs/TransactionUtils/index.ts | 17 +++++++++++------
 src/libs/actions/IOU.ts            |  5 +++--
 src/types/onyx/SearchResults.ts    |  4 ++--
 4 files changed, 22 insertions(+), 25 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 34f68466fe6b..814d1f4a6b52 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -1,3 +1,4 @@
+import type {OnyxCollection} from 'react-native-onyx';
 import type {ValueOf} from 'type-fest';
 import type {SearchColumnType, SearchStatus, SortOrder} from '@components/Search/types';
 import ChatListItem from '@components/SelectionList/ChatListItem';
@@ -10,18 +11,8 @@ import type {TranslationPaths} from '@src/languages/types';
 import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
 import type * as OnyxTypes from '@src/types/onyx';
-import type {TransactionViolation} from '@src/types/onyx';
 import type SearchResults from '@src/types/onyx/SearchResults';
-import type {
-    ListItemDataType,
-    ListItemType,
-    SearchDataTypes,
-    SearchPersonalDetails,
-    SearchPolicy,
-    SearchReport,
-    SearchTransaction,
-    SearchTransactionAction,
-} from '@src/types/onyx/SearchResults';
+import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
 import * as Report from './actions/Report';
 import * as CurrencyUtils from './CurrencyUtils';
@@ -293,7 +284,9 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
             ? data[`${ONYXKEYS.COLLECTION.POLICY}${report?.invoiceReceiver?.policyID}`]
             : undefined;
     const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
-    const allViolations = Object.keys(data).filter((item) => item.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)) ?? {};
+    const allViolations = (Object.entries(data)
+        .filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS))
+        .map((item) => item[1]) ?? undefined) as unknown as OnyxCollection<OnyxTypes.TransactionViolations>;
 
     if (ReportUtils.isSettled(report)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
@@ -311,9 +304,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
 
-    // TODO: update allHavePendingRTERViolation to take in array of transactions
-    // TODO: update hasBrokenConnectionViolation to take in violations instead of connecting directly to Onyx
-    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction)) {
+    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction, allViolations)) {
         return CONST.SEARCH.ACTION_TYPES.SUBMIT;
     }
 
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index 5a96cf3c34c8..715d51b8a387 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -648,8 +648,8 @@ function hasPendingRTERViolation(transactionViolations?: TransactionViolations |
 /**
  * Check if there is broken connection violation.
  */
-function hasBrokenConnectionViolation(transactionID: string): boolean {
-    const violations = getTransactionViolations(transactionID, allTransactionViolations);
+function hasBrokenConnectionViolation(transactionID: string, allViolations?: TransactionViolations): boolean {
+    const violations = allViolations ?? getTransactionViolations(transactionID, allTransactionViolations);
     return !!violations?.find(
         (violation) =>
             violation.name === CONST.VIOLATIONS.RTER &&
@@ -660,9 +660,14 @@ function hasBrokenConnectionViolation(transactionID: string): boolean {
 /**
  * Check if user should see broken connection violation warning.
  */
-function shouldShowBrokenConnectionViolation(transactionID: string, report: OnyxEntry<Report> | SearchReport, policy: OnyxEntry<Policy> | SearchPolicy): boolean {
+function shouldShowBrokenConnectionViolation(
+    transactionID: string,
+    report: OnyxEntry<Report> | SearchReport,
+    policy: OnyxEntry<Policy> | SearchPolicy,
+    allViolations?: TransactionViolations,
+): boolean {
     return (
-        hasBrokenConnectionViolation(transactionID) &&
+        hasBrokenConnectionViolation(transactionID, allViolations) &&
         (!PolicyUtils.isPolicyAdmin(policy) || ReportUtils.isOpenExpenseReport(report) || (ReportUtils.isProcessingReport(report) && PolicyUtils.isInstantSubmitEnabled(policy)))
     );
 }
@@ -670,9 +675,9 @@ function shouldShowBrokenConnectionViolation(transactionID: string, report: Onyx
 /**
  * Check if there is pending rter violation in all transactionViolations with given transactionIDs.
  */
-function allHavePendingRTERViolation(transactionIds: string[]): boolean {
+function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: OnyxCollection<TransactionViolations>): boolean {
     const transactionsWithRTERViolations = transactionIds.map((transactionId) => {
-        const transactionViolations = getTransactionViolations(transactionId, allTransactionViolations);
+        const transactionViolations = getTransactionViolations(transactionId, searchTransactionViolations ?? allTransactionViolations);
         return hasPendingRTERViolation(transactionViolations);
     });
     return transactionsWithRTERViolations.length > 0 && transactionsWithRTERViolations.every((value) => value === true);
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index b7087c91a0db..d7040247c7a2 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7112,11 +7112,12 @@ function canReportBeSubmitted(
     policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
     currentUserAccountID: number,
     transaction?: OnyxTypes.Transaction | SearchTransaction,
+    violations?: OnyxCollection<OnyxTypes.TransactionViolations>,
 ) {
     const isDraft = ReportUtils.isOpenExpenseReport(moneyRequestReport);
     const {reimbursableSpend} = ReportUtils.getMoneyRequestSpendBreakdown(moneyRequestReport);
-    const hasAllPendingRTERViolations = TransactionUtils.allHavePendingRTERViolation([transaction?.transactionID ?? '-1']);
-    const shouldShowBrokenConnectionViolation = TransactionUtils.shouldShowBrokenConnectionViolation(transaction?.transactionID ?? '-1', moneyRequestReport, policy);
+    const hasAllPendingRTERViolations = TransactionUtils.allHavePendingRTERViolation([transaction?.transactionID ?? '-1'], violations);
+    const shouldShowBrokenConnectionViolation = TransactionUtils.shouldShowBrokenConnectionViolation(transaction?.transactionID ?? '-1', moneyRequestReport, policy, violations);
     const isAdmin = policy?.role === CONST.POLICY.ROLE.ADMIN;
 
     return (
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index 117eaf79c598..22270497e6a5 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -8,7 +8,7 @@ import type CONST from '@src/CONST';
 import type ONYXKEYS from '@src/ONYXKEYS';
 import type {InvoiceReceiver} from './Report';
 import type ReportActionName from './ReportActionName';
-import type {TransactionViolation} from './TransactionViolation';
+import type TransactionViolations from './TransactionViolation';
 import ReportNameValuePairs from './ReportNameValuePairs';
 
 /** Types of search data */
@@ -355,7 +355,7 @@ type SearchResults = {
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT, SearchReport> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.POLICY, SearchPolicy> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_NAME_VALUE_PAIRS, ReportNameValuePairs> &
-        PrefixedRecord<typeof ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS, TransactionViolation>;
+        PrefixedRecord<typeof ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS, TransactionViolations>;
 
     /** Whether search data is being fetched from server */
     isLoading?: boolean;

From 69e1aa9354929ad2f51d8f19e2752ed46da3688a Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Tue, 29 Oct 2024 15:41:30 -0600
Subject: [PATCH 25/49] add review action

---
 src/libs/SearchUIUtils.ts | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 814d1f4a6b52..594fdbb91de5 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -283,7 +283,6 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         ReportUtils.isInvoiceReport(report) && report?.invoiceReceiver?.type === CONST.REPORT.INVOICE_RECEIVER_TYPE.BUSINESS
             ? data[`${ONYXKEYS.COLLECTION.POLICY}${report?.invoiceReceiver?.policyID}`]
             : undefined;
-    const violations = data[`${ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS}${transaction?.transactionID}`] ?? {};
     const allViolations = (Object.entries(data)
         .filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS))
         .map((item) => item[1]) ?? undefined) as unknown as OnyxCollection<OnyxTypes.TransactionViolations>;
@@ -308,10 +307,10 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.SUBMIT;
     }
 
-    // const hasViolations = isTransactionEntry(key) ? TransactionUtils.hasViolation(transaction.transactionID, violations) : ReportUtils.hasViolations(report.reportID, allViolations);
-    // if (hasViolations) {
-    //     return CONST.SEARCH.ACTION_TYPES.REVIEW;
-    // }
+    const hasViolations = transaction ? TransactionUtils.hasViolation(transaction.transactionID, allViolations) : ReportUtils.hasViolations(report.reportID, allViolations);
+    if (hasViolations) {
+        return CONST.SEARCH.ACTION_TYPES.REVIEW;
+    }
 
     return CONST.SEARCH.ACTION_TYPES.VIEW;
 }

From 6f9d848b4eccf293a0a4d87a4cab270062effc93 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 12:10:48 -0600
Subject: [PATCH 26/49] fix prettier, lint

---
 src/components/Search/SearchPageHeader.tsx | 2 +-
 src/libs/PolicyUtils.ts                    | 2 +-
 src/libs/TransactionUtils/index.ts         | 2 +-
 src/libs/actions/IOU.ts                    | 6 +++++-
 src/types/onyx/SearchResults.ts            | 2 +-
 5 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/components/Search/SearchPageHeader.tsx b/src/components/Search/SearchPageHeader.tsx
index 5665909185c4..9fb19ff16288 100644
--- a/src/components/Search/SearchPageHeader.tsx
+++ b/src/components/Search/SearchPageHeader.tsx
@@ -182,7 +182,7 @@ function SearchPageHeader({queryJSON, hash}: SearchPageHeaderProps) {
                     return;
                 }
 
-                const reportIDList = (selectedReports?.filter((report) => !!report) as string[]) ?? [];
+                const reportIDList = selectedReports?.filter((report) => !!report) ?? [];
                 SearchActions.exportSearchItemsToCSV(
                     {query: status, jsonQuery: JSON.stringify(queryJSON), reportIDList, transactionIDList: selectedTransactionsKeys, policyIDs: [activeWorkspaceID ?? '']},
                     () => {
diff --git a/src/libs/PolicyUtils.ts b/src/libs/PolicyUtils.ts
index 0f018d8583a9..0cc833f69f82 100644
--- a/src/libs/PolicyUtils.ts
+++ b/src/libs/PolicyUtils.ts
@@ -29,7 +29,7 @@ import type {
     Tenant,
 } from '@src/types/onyx/Policy';
 import type PolicyEmployee from '@src/types/onyx/PolicyEmployee';
-import {SearchPolicy} from '@src/types/onyx/SearchResults';
+import type {SearchPolicy} from '@src/types/onyx/SearchResults';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
 import {hasSynchronizationErrorMessage} from './actions/connections';
 import * as Localize from './Localize';
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index 715d51b8a387..b1dcbfa5f5a9 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -24,7 +24,7 @@ import type {IOUType} from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import type {Beta, OnyxInputOrEntry, Policy, RecentWaypoint, Report, ReviewDuplicates, TaxRate, TaxRates, Transaction, TransactionViolation, TransactionViolations} from '@src/types/onyx';
 import type {Attendee} from '@src/types/onyx/IOU';
-import {SearchPolicy, SearchReport} from '@src/types/onyx/SearchResults';
+import type {SearchPolicy, SearchReport} from '@src/types/onyx/SearchResults';
 import type {Comment, Receipt, TransactionChanges, TransactionPendingFieldsKey, Waypoint, WaypointCollection} from '@src/types/onyx/Transaction';
 import type DeepValueOf from '@src/types/utils/DeepValueOf';
 import {isEmptyObject} from '@src/types/utils/EmptyObject';
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index d7040247c7a2..812cebe08713 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7016,7 +7016,11 @@ function sendMoneyWithWallet(report: OnyxEntry<OnyxTypes.Report>, amount: number
     Report.notifyNewAction(params.chatReportID, managerID);
 }
 
-function canApproveIOU(iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport, policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy, chatReportRNVP?: OnyxTypes.ReportNameValuePairs) {
+function canApproveIOU(
+    iouReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
+    policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
+    chatReportRNVP?: OnyxTypes.ReportNameValuePairs,
+) {
     // Only expense reports can be approved
     const isPaidGroupPolicy = policy && PolicyUtils.isPaidGroupPolicy(policy);
     if (!isPaidGroupPolicy) {
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index 22270497e6a5..26d258a0142a 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -8,8 +8,8 @@ import type CONST from '@src/CONST';
 import type ONYXKEYS from '@src/ONYXKEYS';
 import type {InvoiceReceiver} from './Report';
 import type ReportActionName from './ReportActionName';
+import type ReportNameValuePairs from './ReportNameValuePairs';
 import type TransactionViolations from './TransactionViolation';
-import ReportNameValuePairs from './ReportNameValuePairs';
 
 /** Types of search data */
 type SearchDataTypes = ValueOf<typeof CONST.SEARCH.DATA_TYPES>;

From 65b00e322c6ace76fc11e238334d27e9933c7c36 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 12:30:07 -0600
Subject: [PATCH 27/49] fix types

---
 src/libs/SearchUIUtils.ts          | 6 +++---
 src/libs/TransactionUtils/index.ts | 4 ++--
 src/libs/actions/IOU.ts            | 6 +++---
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 594fdbb91de5..dd5abfae40b6 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -262,7 +262,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     const transaction = isTransactionEntry(key) ? data[key] : undefined;
-    const report = (transaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key]) as SearchReport;
+    const report = isReportEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : (data[key] as SearchReport);
 
     // We don't need to run the logic if this is not a transaction or iou/expense report
     if (!ReportUtils.isMoneyRequestReport(report)) {
@@ -283,9 +283,9 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         ReportUtils.isInvoiceReport(report) && report?.invoiceReceiver?.type === CONST.REPORT.INVOICE_RECEIVER_TYPE.BUSINESS
             ? data[`${ONYXKEYS.COLLECTION.POLICY}${report?.invoiceReceiver?.policyID}`]
             : undefined;
-    const allViolations = (Object.entries(data)
+    const allViolations = Object.entries(data)
         .filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS))
-        .map((item) => item[1]) ?? undefined) as unknown as OnyxCollection<OnyxTypes.TransactionViolations>;
+        .map((item) => item[1]);
 
     if (ReportUtils.isSettled(report)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index b1dcbfa5f5a9..dba0ff7b25cf 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -675,9 +675,9 @@ function shouldShowBrokenConnectionViolation(
 /**
  * Check if there is pending rter violation in all transactionViolations with given transactionIDs.
  */
-function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: OnyxCollection<TransactionViolations>): boolean {
+function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: TransactionViolations): boolean {
     const transactionsWithRTERViolations = transactionIds.map((transactionId) => {
-        const transactionViolations = getTransactionViolations(transactionId, searchTransactionViolations ?? allTransactionViolations);
+        const transactionViolations = searchTransactionViolations ?? getTransactionViolations(transactionId, allTransactionViolations);
         return hasPendingRTERViolation(transactionViolations);
     });
     return transactionsWithRTERViolations.length > 0 && transactionsWithRTERViolations.every((value) => value === true);
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index 812cebe08713..c6f58d8352c6 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7112,11 +7112,11 @@ function canIOUBePaid(
 }
 
 function canReportBeSubmitted(
-    moneyRequestReport: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Report> | SearchReport,
-    policy: OnyxTypes.OnyxInputOrEntry<OnyxTypes.Policy> | SearchPolicy,
+    moneyRequestReport: OnyxEntry<OnyxTypes.Report> | SearchReport,
+    policy: OnyxEntry<OnyxTypes.Policy> | SearchPolicy,
     currentUserAccountID: number,
     transaction?: OnyxTypes.Transaction | SearchTransaction,
-    violations?: OnyxCollection<OnyxTypes.TransactionViolations>,
+    violations?: OnyxTypes.TransactionViolations,
 ) {
     const isDraft = ReportUtils.isOpenExpenseReport(moneyRequestReport);
     const {reimbursableSpend} = ReportUtils.getMoneyRequestSpendBreakdown(moneyRequestReport);

From d259dc864b04835b3cedaf84ea29e80f712234b0 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 12:32:24 -0600
Subject: [PATCH 28/49] fix more types

---
 src/libs/SearchUIUtils.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index dd5abfae40b6..cc9869582e8c 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -1,4 +1,3 @@
-import type {OnyxCollection} from 'react-native-onyx';
 import type {ValueOf} from 'type-fest';
 import type {SearchColumnType, SearchStatus, SortOrder} from '@components/Search/types';
 import ChatListItem from '@components/SelectionList/ChatListItem';

From 7919753e927f5d5699b3f11b75c9813f5486de63 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 13:03:58 -0600
Subject: [PATCH 29/49] fix violations types

---
 src/libs/SearchUIUtils.ts          | 13 +++++++------
 src/libs/TransactionUtils/index.ts | 10 +++++-----
 src/libs/actions/IOU.ts            |  2 +-
 3 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index cc9869582e8c..1e22170f3946 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -21,6 +21,7 @@ import Navigation from './Navigation/Navigation';
 import * as ReportActionsUtils from './ReportActionsUtils';
 import * as ReportUtils from './ReportUtils';
 import * as TransactionUtils from './TransactionUtils';
+import { OnyxCollection } from 'react-native-onyx';
 
 const columnNamesToSortingProperty = {
     [CONST.SEARCH.TABLE_COLUMNS.TO]: 'formattedTo' as const,
@@ -261,7 +262,7 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
     }
 
     const transaction = isTransactionEntry(key) ? data[key] : undefined;
-    const report = isReportEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : (data[key] as SearchReport);
+    const report = isTransactionEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key];
 
     // We don't need to run the logic if this is not a transaction or iou/expense report
     if (!ReportUtils.isMoneyRequestReport(report)) {
@@ -282,9 +283,9 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         ReportUtils.isInvoiceReport(report) && report?.invoiceReceiver?.type === CONST.REPORT.INVOICE_RECEIVER_TYPE.BUSINESS
             ? data[`${ONYXKEYS.COLLECTION.POLICY}${report?.invoiceReceiver?.policyID}`]
             : undefined;
-    const allViolations = Object.entries(data)
-        .filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS))
-        .map((item) => item[1]);
+    const violationsCollection = Object.fromEntries(
+        Object.entries(data).filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)),
+    ) as unknown as OnyxCollection<OnyxTypes.TransactionViolations>;
 
     if (ReportUtils.isSettled(report)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
@@ -302,11 +303,11 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
 
-    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction, allViolations)) {
+    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction, violationsCollection)) {
         return CONST.SEARCH.ACTION_TYPES.SUBMIT;
     }
 
-    const hasViolations = transaction ? TransactionUtils.hasViolation(transaction.transactionID, allViolations) : ReportUtils.hasViolations(report.reportID, allViolations);
+    const hasViolations = transaction ? TransactionUtils.hasViolation(transaction.transactionID, violationsCollection) : ReportUtils.hasViolations(report.reportID, violationsCollection);
     if (hasViolations) {
         return CONST.SEARCH.ACTION_TYPES.REVIEW;
     }
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index dba0ff7b25cf..f8f9f9e9dc7e 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -648,8 +648,8 @@ function hasPendingRTERViolation(transactionViolations?: TransactionViolations |
 /**
  * Check if there is broken connection violation.
  */
-function hasBrokenConnectionViolation(transactionID: string, allViolations?: TransactionViolations): boolean {
-    const violations = allViolations ?? getTransactionViolations(transactionID, allTransactionViolations);
+function hasBrokenConnectionViolation(transactionID: string, allViolations?: OnyxCollection<TransactionViolations>): boolean {
+    const violations = getTransactionViolations(transactionID, allViolations ?? allTransactionViolations);
     return !!violations?.find(
         (violation) =>
             violation.name === CONST.VIOLATIONS.RTER &&
@@ -664,7 +664,7 @@ function shouldShowBrokenConnectionViolation(
     transactionID: string,
     report: OnyxEntry<Report> | SearchReport,
     policy: OnyxEntry<Policy> | SearchPolicy,
-    allViolations?: TransactionViolations,
+    allViolations?: OnyxCollection<TransactionViolations>,
 ): boolean {
     return (
         hasBrokenConnectionViolation(transactionID, allViolations) &&
@@ -675,9 +675,9 @@ function shouldShowBrokenConnectionViolation(
 /**
  * Check if there is pending rter violation in all transactionViolations with given transactionIDs.
  */
-function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: TransactionViolations): boolean {
+function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: OnyxCollection<TransactionViolations>): boolean {
     const transactionsWithRTERViolations = transactionIds.map((transactionId) => {
-        const transactionViolations = searchTransactionViolations ?? getTransactionViolations(transactionId, allTransactionViolations);
+        const transactionViolations = getTransactionViolations(transactionId, searchTransactionViolations ?? allTransactionViolations);
         return hasPendingRTERViolation(transactionViolations);
     });
     return transactionsWithRTERViolations.length > 0 && transactionsWithRTERViolations.every((value) => value === true);
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index c6f58d8352c6..d3b54cb4e1aa 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7116,7 +7116,7 @@ function canReportBeSubmitted(
     policy: OnyxEntry<OnyxTypes.Policy> | SearchPolicy,
     currentUserAccountID: number,
     transaction?: OnyxTypes.Transaction | SearchTransaction,
-    violations?: OnyxTypes.TransactionViolations,
+    violations?: OnyxCollection<OnyxTypes.TransactionViolations>,
 ) {
     const isDraft = ReportUtils.isOpenExpenseReport(moneyRequestReport);
     const {reimbursableSpend} = ReportUtils.getMoneyRequestSpendBreakdown(moneyRequestReport);

From 88f199eb28dbc1d3d524df52a05365d368043465 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 13:41:32 -0600
Subject: [PATCH 30/49] fix lint

---
 src/libs/SearchUIUtils.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 1e22170f3946..6ee352657879 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -1,3 +1,4 @@
+import type {OnyxCollection} from 'react-native-onyx';
 import type {ValueOf} from 'type-fest';
 import type {SearchColumnType, SearchStatus, SortOrder} from '@components/Search/types';
 import ChatListItem from '@components/SelectionList/ChatListItem';
@@ -21,7 +22,6 @@ import Navigation from './Navigation/Navigation';
 import * as ReportActionsUtils from './ReportActionsUtils';
 import * as ReportUtils from './ReportUtils';
 import * as TransactionUtils from './TransactionUtils';
-import { OnyxCollection } from 'react-native-onyx';
 
 const columnNamesToSortingProperty = {
     [CONST.SEARCH.TABLE_COLUMNS.TO]: 'formattedTo' as const,

From c5b87404d999f1ee1d687b5d5e9f69799dc73d76 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 14:15:13 -0600
Subject: [PATCH 31/49] refactor code for performance

---
 src/libs/SearchUIUtils.ts | 47 ++++++++++++++++++++-------------------
 1 file changed, 24 insertions(+), 23 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 6ee352657879..c53638b61aa6 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -253,39 +253,21 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
  * Do not use directly, use only via `getSections()` facade.
  */
 function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUserAccountID: number): SearchTransactionAction {
-    if (!isTransactionEntry(key) && !isReportEntry(key)) {
+    const isTransaction = isTransactionEntry(key);
+    if ((!isTransaction && !isReportEntry(key)) || (isTransaction && !data[key].isFromOneTransactionReport)) {
         return CONST.SEARCH.ACTION_TYPES.VIEW;
     }
 
-    if (isTransactionEntry(key) && !data[key].isFromOneTransactionReport) {
-        return CONST.SEARCH.ACTION_TYPES.VIEW;
-    }
-
-    const transaction = isTransactionEntry(key) ? data[key] : undefined;
-    const report = isTransactionEntry(key) ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key];
+    const transaction = isTransaction ? data[key] : undefined;
+    const report = isTransaction ? data[`${ONYXKEYS.COLLECTION.REPORT}${transaction?.reportID}`] : data[key];
 
-    // We don't need to run the logic if this is not a transaction or iou/expense report
+    // We don't need to run the logic if this is not a transaction or iou/expense report, so let's shortcircuit the logic for performance reasons
     if (!ReportUtils.isMoneyRequestReport(report)) {
         return CONST.SEARCH.ACTION_TYPES.VIEW;
     }
 
     const chatReport = data[`${ONYXKEYS.COLLECTION.REPORT}${report?.chatReportID}`] ?? {};
     const chatReportRNVP = data[`${ONYXKEYS.COLLECTION.REPORT_NAME_VALUE_PAIRS}${report?.chatReportID}`] ?? undefined;
-    const allReportTransactions = (
-        isReportEntry(key)
-            ? Object.entries(data)
-                  .filter(([itemKey, value]) => isTransactionEntry(itemKey) && (value as SearchTransaction)?.reportID === report.reportID)
-                  .map((item) => item[1])
-            : []
-    ) as SearchTransaction[];
-    const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`] ?? {};
-    const invoiceReceiverPolicy =
-        ReportUtils.isInvoiceReport(report) && report?.invoiceReceiver?.type === CONST.REPORT.INVOICE_RECEIVER_TYPE.BUSINESS
-            ? data[`${ONYXKEYS.COLLECTION.POLICY}${report?.invoiceReceiver?.policyID}`]
-            : undefined;
-    const violationsCollection = Object.fromEntries(
-        Object.entries(data).filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)),
-    ) as unknown as OnyxCollection<OnyxTypes.TransactionViolations>;
 
     if (ReportUtils.isSettled(report)) {
         return CONST.SEARCH.ACTION_TYPES.PAID;
@@ -295,6 +277,21 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.DONE;
     }
 
+    const policy = data[`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`] ?? {};
+
+    const invoiceReceiverPolicy =
+        ReportUtils.isInvoiceReport(report) && report?.invoiceReceiver?.type === CONST.REPORT.INVOICE_RECEIVER_TYPE.BUSINESS
+            ? data[`${ONYXKEYS.COLLECTION.POLICY}${report?.invoiceReceiver?.policyID}`]
+            : undefined;
+
+    const allReportTransactions = (
+        isReportEntry(key)
+            ? Object.entries(data)
+                  .filter(([itemKey, value]) => isTransactionEntry(itemKey) && (value as SearchTransaction)?.reportID === report.reportID)
+                  .map((item) => item[1])
+            : []
+    ) as SearchTransaction[];
+
     if (IOU.canIOUBePaid(report, chatReport, policy, allReportTransactions, false, chatReportRNVP, invoiceReceiverPolicy)) {
         return CONST.SEARCH.ACTION_TYPES.PAY;
     }
@@ -303,6 +300,10 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
 
+    const violationsCollection = Object.fromEntries(
+        Object.entries(data).filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)),
+    ) as unknown as OnyxCollection<OnyxTypes.TransactionViolations>;
+
     if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction, violationsCollection)) {
         return CONST.SEARCH.ACTION_TYPES.SUBMIT;
     }

From 17ce03d32bd091c8272b4a7bc59a2b622d13e9f2 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 14:38:34 -0600
Subject: [PATCH 32/49] rm violations code since its not ready yet

---
 src/libs/SearchUIUtils.ts          | 13 -------------
 src/libs/TransactionUtils/index.ts |  1 +
 2 files changed, 1 insertion(+), 13 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index c53638b61aa6..5df592c0acef 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -300,19 +300,6 @@ function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUs
         return CONST.SEARCH.ACTION_TYPES.APPROVE;
     }
 
-    const violationsCollection = Object.fromEntries(
-        Object.entries(data).filter(([itemKey]) => itemKey.startsWith(ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS)),
-    ) as unknown as OnyxCollection<OnyxTypes.TransactionViolations>;
-
-    if (IOU.canReportBeSubmitted(report, policy, currentUserAccountID, transaction, violationsCollection)) {
-        return CONST.SEARCH.ACTION_TYPES.SUBMIT;
-    }
-
-    const hasViolations = transaction ? TransactionUtils.hasViolation(transaction.transactionID, violationsCollection) : ReportUtils.hasViolations(report.reportID, violationsCollection);
-    if (hasViolations) {
-        return CONST.SEARCH.ACTION_TYPES.REVIEW;
-    }
-
     return CONST.SEARCH.ACTION_TYPES.VIEW;
 }
 
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index f8f9f9e9dc7e..8935b8d8ec54 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -678,6 +678,7 @@ function shouldShowBrokenConnectionViolation(
 function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: OnyxCollection<TransactionViolations>): boolean {
     const transactionsWithRTERViolations = transactionIds.map((transactionId) => {
         const transactionViolations = getTransactionViolations(transactionId, searchTransactionViolations ?? allTransactionViolations);
+        console.log('over here', transactionViolations)
         return hasPendingRTERViolation(transactionViolations);
     });
     return transactionsWithRTERViolations.length > 0 && transactionsWithRTERViolations.every((value) => value === true);

From 01aa98f0eaa0de6b0697685ab3da9956fc34ac92 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 14:49:10 -0600
Subject: [PATCH 33/49] add callbacks

---
 src/components/SelectionList/Search/ReportListItem.tsx | 9 ++++++++-
 src/libs/TransactionUtils/index.ts                     | 1 -
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/components/SelectionList/Search/ReportListItem.tsx b/src/components/SelectionList/Search/ReportListItem.tsx
index 4a88c9ef9b26..6ed9a7831583 100644
--- a/src/components/SelectionList/Search/ReportListItem.tsx
+++ b/src/components/SelectionList/Search/ReportListItem.tsx
@@ -94,7 +94,14 @@ function ReportListItem<TItem extends ListItem>({
     ];
 
     const handleOnButtonPress = () => {
-        onSelectRow(item);
+        switch (reportItem.action) {
+            case CONST.SEARCH.ACTION_TYPES.APPROVE:
+                return console.log('approve');
+            case CONST.SEARCH.ACTION_TYPES.PAY:
+                return console.log('pay');
+            default:
+                return onSelectRow(item);
+        }
     };
 
     const openReportInRHP = (transactionItem: TransactionListItemType) => {
diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index 8935b8d8ec54..f8f9f9e9dc7e 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -678,7 +678,6 @@ function shouldShowBrokenConnectionViolation(
 function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: OnyxCollection<TransactionViolations>): boolean {
     const transactionsWithRTERViolations = transactionIds.map((transactionId) => {
         const transactionViolations = getTransactionViolations(transactionId, searchTransactionViolations ?? allTransactionViolations);
-        console.log('over here', transactionViolations)
         return hasPendingRTERViolation(transactionViolations);
     });
     return transactionsWithRTERViolations.length > 0 && transactionsWithRTERViolations.every((value) => value === true);

From 0ad354c96b75e7f9ebc635119757e5ee2d377484 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 17:27:58 -0600
Subject: [PATCH 34/49] create action to handle callbacks

---
 .../SelectionList/Search/ReportListItem.tsx   | 12 +++----
 src/libs/actions/Search.ts                    | 36 ++++++++++++++-----
 2 files changed, 32 insertions(+), 16 deletions(-)

diff --git a/src/components/SelectionList/Search/ReportListItem.tsx b/src/components/SelectionList/Search/ReportListItem.tsx
index 6ed9a7831583..20a9970e77c3 100644
--- a/src/components/SelectionList/Search/ReportListItem.tsx
+++ b/src/components/SelectionList/Search/ReportListItem.tsx
@@ -1,6 +1,7 @@
 import React from 'react';
 import {View} from 'react-native';
 import Checkbox from '@components/Checkbox';
+import {useSearchContext} from '@components/Search/SearchContext';
 import BaseListItem from '@components/SelectionList/BaseListItem';
 import type {ListItem, ReportListItemProps, ReportListItemType, TransactionListItemType} from '@components/SelectionList/types';
 import Text from '@components/Text';
@@ -10,6 +11,7 @@ import useResponsiveLayout from '@hooks/useResponsiveLayout';
 import useStyleUtils from '@hooks/useStyleUtils';
 import useTheme from '@hooks/useTheme';
 import useThemeStyles from '@hooks/useThemeStyles';
+import {handleActionButtonPress} from '@libs/actions/Search';
 import * as CurrencyUtils from '@libs/CurrencyUtils';
 import Navigation from '@libs/Navigation/Navigation';
 import variables from '@styles/variables';
@@ -69,6 +71,7 @@ function ReportListItem<TItem extends ListItem>({
     const styles = useThemeStyles();
     const {isLargeScreenWidth} = useResponsiveLayout();
     const StyleUtils = useStyleUtils();
+    const {currentSearchHash} = useSearchContext();
 
     const animatedHighlightStyle = useAnimatedHighlightStyle({
         borderRadius: variables.componentBorderRadius,
@@ -94,14 +97,7 @@ function ReportListItem<TItem extends ListItem>({
     ];
 
     const handleOnButtonPress = () => {
-        switch (reportItem.action) {
-            case CONST.SEARCH.ACTION_TYPES.APPROVE:
-                return console.log('approve');
-            case CONST.SEARCH.ACTION_TYPES.PAY:
-                return console.log('pay');
-            default:
-                return onSelectRow(item);
-        }
+        handleActionButtonPress(currentSearchHash, reportItem, () => onSelectRow(item));
     };
 
     const openReportInRHP = (transactionItem: TransactionListItemType) => {
diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index caef871571ed..3fe073d7f944 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -1,5 +1,5 @@
 import Onyx from 'react-native-onyx';
-import type {OnyxUpdate} from 'react-native-onyx';
+import type {OnyxEntry, OnyxUpdate} from 'react-native-onyx';
 import type {ValueOf} from 'type-fest';
 import type {FormOnyxValues} from '@components/Form/types';
 import type {SearchQueryJSON} from '@components/Search/types';
@@ -12,7 +12,8 @@ import enhanceParameters from '@libs/Network/enhanceParameters';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import FILTER_KEYS from '@src/types/form/SearchAdvancedFiltersForm';
-import type {SearchTransaction} from '@src/types/onyx/SearchResults';
+import type {LastPaymentMethod} from '@src/types/onyx';
+import type {SearchReport, SearchTransaction} from '@src/types/onyx/SearchResults';
 import * as Report from './Report';
 
 let currentUserEmail: string;
@@ -23,6 +24,26 @@ Onyx.connect({
     },
 });
 
+let lastPaymentMethod: OnyxEntry<LastPaymentMethod>;
+Onyx.connect({
+    key: ONYXKEYS.NVP_LAST_PAYMENT_METHOD,
+    callback: (val) => {
+        lastPaymentMethod = val;
+    },
+});
+
+function handleActionButtonPress(hash: number, item: SearchTransaction | SearchReport, callback: () => void) {
+    const lastPolicyPaymentMethod = item.policyID ? lastPaymentMethod?.policyID : null;
+    switch (item.action) {
+        case CONST.SEARCH.ACTION_TYPES.PAY:
+            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, lastPolicyPaymentMethod, {[item.reportID]: 0}) : callback();
+        case CONST.SEARCH.ACTION_TYPES.APPROVE:
+            return approveMoneyRequestOnSearch(hash, [item.reportID]);
+        default:
+            return callback();
+    }
+}
+
 function getOnyxLoadingData(hash: number): {optimisticData: OnyxUpdate[]; finallyData: OnyxUpdate[]} {
     const optimisticData: OnyxUpdate[] = [
         {
@@ -164,20 +185,16 @@ function holdMoneyRequestOnSearch(hash: number, transactionIDList: string[], com
     API.write(WRITE_COMMANDS.HOLD_MONEY_REQUEST_ON_SEARCH, {hash, transactionIDList, comment}, {optimisticData, finallyData});
 }
 
-// this function will be used once https://github.com/Expensify/App/pull/51445 is merged
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
 function approveMoneyRequestOnSearch(hash: number, reportIDList: string[]) {
     const {optimisticData, finallyData} = getOnyxLoadingData(hash);
 
     API.write(WRITE_COMMANDS.APPROVE_MONEY_REQUEST_ON_SEARCH, {hash, reportIDList}, {optimisticData, finallyData});
 }
 
-// this function will be used once https://github.com/Expensify/App/pull/51445 is merged
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
-function payMoneyRequestOnSearch(hash: number, paymentType: string, reportsAndAmounts: string) {
+function payMoneyRequestOnSearch(hash: number, paymentType: string, reportsAndAmounts: Record<string, number>) {
     const {optimisticData, finallyData} = getOnyxLoadingData(hash);
 
-    API.write(WRITE_COMMANDS.PAY_MONEY_REQUEST_ON_SEARCH, {hash, paymentType, reportsAndAmounts}, {optimisticData, finallyData});
+    API.write(WRITE_COMMANDS.PAY_MONEY_REQUEST_ON_SEARCH, {hash, paymentType, reportsAndAmounts: JSON.stringify(reportsAndAmounts)}, {optimisticData, finallyData});
 }
 
 function unholdMoneyRequestOnSearch(hash: number, transactionIDList: string[]) {
@@ -261,4 +278,7 @@ export {
     deleteSavedSearch,
     dismissSavedSearchRenameTooltip,
     showSavedSearchRenameTooltip,
+    payMoneyRequestOnSearch,
+    approveMoneyRequestOnSearch,
+    handleActionButtonPress,
 };

From 89f4ebe0b5dd67e92633e38ccb220476d12314ba Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Wed, 30 Oct 2024 17:29:32 -0600
Subject: [PATCH 35/49] add callback to transaction list item

---
 src/components/SelectionList/Search/TransactionListItem.tsx | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/components/SelectionList/Search/TransactionListItem.tsx b/src/components/SelectionList/Search/TransactionListItem.tsx
index 4fd1ec4cce75..d5df46cb2e19 100644
--- a/src/components/SelectionList/Search/TransactionListItem.tsx
+++ b/src/components/SelectionList/Search/TransactionListItem.tsx
@@ -1,10 +1,12 @@
 import React from 'react';
+import {useSearchContext} from '@components/Search/SearchContext';
 import BaseListItem from '@components/SelectionList/BaseListItem';
 import type {ListItem, TransactionListItemProps, TransactionListItemType} from '@components/SelectionList/types';
 import useAnimatedHighlightStyle from '@hooks/useAnimatedHighlightStyle';
 import useResponsiveLayout from '@hooks/useResponsiveLayout';
 import useTheme from '@hooks/useTheme';
 import useThemeStyles from '@hooks/useThemeStyles';
+import {handleActionButtonPress} from '@libs/actions/Search';
 import variables from '@styles/variables';
 import TransactionListItemRow from './TransactionListItemRow';
 
@@ -26,6 +28,7 @@ function TransactionListItem<TItem extends ListItem>({
     const theme = useTheme();
 
     const {isLargeScreenWidth} = useResponsiveLayout();
+    const {currentSearchHash} = useSearchContext();
 
     const listItemPressableStyle = [
         styles.selectionListPressableItemWrapper,
@@ -76,7 +79,7 @@ function TransactionListItem<TItem extends ListItem>({
                 item={transactionItem}
                 showTooltip={showTooltip}
                 onButtonPress={() => {
-                    onSelectRow(item);
+                    handleActionButtonPress(currentSearchHash, transactionItem, () => onSelectRow(item));
                 }}
                 onCheckboxPress={() => onCheckboxPress?.(item)}
                 isDisabled={!!isDisabled}

From 4a4481fdbf0d889929b3650da20b2d7348083a54 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 31 Oct 2024 14:29:38 -0600
Subject: [PATCH 36/49] rename param

---
 src/libs/actions/Search.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index 3fe073d7f944..b3ae8b1d8faf 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -32,15 +32,15 @@ Onyx.connect({
     },
 });
 
-function handleActionButtonPress(hash: number, item: SearchTransaction | SearchReport, callback: () => void) {
+function handleActionButtonPress(hash: number, item: SearchTransaction | SearchReport, goToItem: () => void) {
     const lastPolicyPaymentMethod = item.policyID ? lastPaymentMethod?.policyID : null;
     switch (item.action) {
         case CONST.SEARCH.ACTION_TYPES.PAY:
-            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, lastPolicyPaymentMethod, {[item.reportID]: 0}) : callback();
+            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, lastPolicyPaymentMethod, {[item.reportID]: 0}) : goToItem();
         case CONST.SEARCH.ACTION_TYPES.APPROVE:
             return approveMoneyRequestOnSearch(hash, [item.reportID]);
         default:
-            return callback();
+            return goToItem();
     }
 }
 

From ee12a893b7ac890d4eab0c6c86113ed00ab12432 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 31 Oct 2024 14:30:57 -0600
Subject: [PATCH 37/49] fix payment method

---
 src/libs/actions/Search.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index b3ae8b1d8faf..9395e58d4099 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -33,7 +33,7 @@ Onyx.connect({
 });
 
 function handleActionButtonPress(hash: number, item: SearchTransaction | SearchReport, goToItem: () => void) {
-    const lastPolicyPaymentMethod = item.policyID ? lastPaymentMethod?.policyID : null;
+    const lastPolicyPaymentMethod = item.policyID ? lastPaymentMethod?.[item.policyID] : null;
     switch (item.action) {
         case CONST.SEARCH.ACTION_TYPES.PAY:
             return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, lastPolicyPaymentMethod, {[item.reportID]: 0}) : goToItem();

From 3ee97f8b231295967545764eea5ed80c428eaae7 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 31 Oct 2024 16:20:07 -0600
Subject: [PATCH 38/49] revert unrelated change

---
 src/components/Search/index.tsx | 2 +-
 src/libs/SearchUIUtils.ts       | 1 -
 src/libs/actions/Search.ts      | 9 ++++++---
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/components/Search/index.tsx b/src/components/Search/index.tsx
index 52ada9fb9328..9238488361b0 100644
--- a/src/components/Search/index.tsx
+++ b/src/components/Search/index.tsx
@@ -249,7 +249,7 @@ function Search({queryJSON, onSearchListScroll, contentContainerStyle}: SearchPr
                 });
             });
         }
-        // setSelectedTransactions(newTransactionList, data);
+        setSelectedTransactions(newTransactionList, data);
         // eslint-disable-next-line react-compiler/react-compiler, react-hooks/exhaustive-deps
     }, [data, setSelectedTransactions]);
 
diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 5df592c0acef..88c417f9ce7e 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -1,4 +1,3 @@
-import type {OnyxCollection} from 'react-native-onyx';
 import type {ValueOf} from 'type-fest';
 import type {SearchColumnType, SearchStatus, SortOrder} from '@components/Search/types';
 import ChatListItem from '@components/SelectionList/ChatListItem';
diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index 9395e58d4099..556fe76df4d1 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -3,17 +3,19 @@ import type {OnyxEntry, OnyxUpdate} from 'react-native-onyx';
 import type {ValueOf} from 'type-fest';
 import type {FormOnyxValues} from '@components/Form/types';
 import type {SearchQueryJSON} from '@components/Search/types';
+import type {ReportListItemType, TransactionListItemType} from '@components/SelectionList/types';
 import * as API from '@libs/API';
 import type {ExportSearchItemsToCSVParams} from '@libs/API/parameters';
 import {WRITE_COMMANDS} from '@libs/API/types';
 import * as ApiUtils from '@libs/ApiUtils';
 import fileDownload from '@libs/fileDownload';
 import enhanceParameters from '@libs/Network/enhanceParameters';
+import {isReportListItemType} from '@libs/SearchUIUtils';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import FILTER_KEYS from '@src/types/form/SearchAdvancedFiltersForm';
 import type {LastPaymentMethod} from '@src/types/onyx';
-import type {SearchReport, SearchTransaction} from '@src/types/onyx/SearchResults';
+import type {SearchTransaction} from '@src/types/onyx/SearchResults';
 import * as Report from './Report';
 
 let currentUserEmail: string;
@@ -32,11 +34,12 @@ Onyx.connect({
     },
 });
 
-function handleActionButtonPress(hash: number, item: SearchTransaction | SearchReport, goToItem: () => void) {
+function handleActionButtonPress(hash: number, item: TransactionListItemType | ReportListItemType, goToItem: () => void) {
     const lastPolicyPaymentMethod = item.policyID ? lastPaymentMethod?.[item.policyID] : null;
+    const amount = isReportListItemType(item) ? item.total ?? 0 : item.formattedTotal;
     switch (item.action) {
         case CONST.SEARCH.ACTION_TYPES.PAY:
-            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, lastPolicyPaymentMethod, {[item.reportID]: 0}) : goToItem();
+            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, lastPolicyPaymentMethod, {[item.reportID]: amount}) : goToItem();
         case CONST.SEARCH.ACTION_TYPES.APPROVE:
             return approveMoneyRequestOnSearch(hash, [item.reportID]);
         default:

From 77ae5ef37340bb480103a6ef91e344dc93b0d91c Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Thu, 31 Oct 2024 16:25:37 -0600
Subject: [PATCH 39/49] fix tscheck

---
 src/libs/SearchUIUtils.ts | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/libs/SearchUIUtils.ts b/src/libs/SearchUIUtils.ts
index 88c417f9ce7e..29537b9c8368 100644
--- a/src/libs/SearchUIUtils.ts
+++ b/src/libs/SearchUIUtils.ts
@@ -13,7 +13,6 @@ import type * as OnyxTypes from '@src/types/onyx';
 import type SearchResults from '@src/types/onyx/SearchResults';
 import type {ListItemDataType, ListItemType, SearchDataTypes, SearchPersonalDetails, SearchReport, SearchTransaction, SearchTransactionAction} from '@src/types/onyx/SearchResults';
 import * as IOU from './actions/IOU';
-import * as Report from './actions/Report';
 import * as CurrencyUtils from './CurrencyUtils';
 import DateUtils from './DateUtils';
 import {translateLocal} from './Localize';
@@ -212,9 +211,7 @@ function getIOUReportName(data: OnyxTypes.SearchResults['data'], reportItem: Sea
  */
 function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata: OnyxTypes.SearchResults['search']): TransactionListItemType[] {
     const shouldShowMerchant = getShouldShowMerchant(data);
-
     const doesDataContainAPastYearTransaction = shouldShowYear(data);
-    const currentUserAccountID = Report.getCurrentUserAccountID();
 
     return Object.keys(data)
         .filter(isTransactionEntry)
@@ -227,7 +224,7 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
 
             return {
                 ...transactionItem,
-                action: getAction(data, key, currentUserAccountID),
+                action: getAction(data, key),
                 from,
                 to,
                 formattedFrom,
@@ -251,7 +248,7 @@ function getTransactionsSections(data: OnyxTypes.SearchResults['data'], metadata
  *
  * Do not use directly, use only via `getSections()` facade.
  */
-function getAction(data: OnyxTypes.SearchResults['data'], key: string, currentUserAccountID: number): SearchTransactionAction {
+function getAction(data: OnyxTypes.SearchResults['data'], key: string): SearchTransactionAction {
     const isTransaction = isTransactionEntry(key);
     if ((!isTransaction && !isReportEntry(key)) || (isTransaction && !data[key].isFromOneTransactionReport)) {
         return CONST.SEARCH.ACTION_TYPES.VIEW;
@@ -339,7 +336,6 @@ function getReportActionsSections(data: OnyxTypes.SearchResults['data']): Report
  * Do not use directly, use only via `getSections()` facade.
  */
 function getReportSections(data: OnyxTypes.SearchResults['data'], metadata: OnyxTypes.SearchResults['search']): ReportListItemType[] {
-    const currentUserAccountID = Report.getCurrentUserAccountID();
     const shouldShowMerchant = getShouldShowMerchant(data);
 
     const doesDataContainAPastYearTransaction = shouldShowYear(data);
@@ -354,7 +350,7 @@ function getReportSections(data: OnyxTypes.SearchResults['data'], metadata: Onyx
 
             reportIDToTransactions[reportKey] = {
                 ...reportItem,
-                action: getAction(data, key, currentUserAccountID),
+                action: getAction(data, key),
                 keyForList: reportItem.reportID,
                 from: data.personalDetailsList?.[reportItem.accountID ?? -1],
                 to: reportItem.managerID ? data.personalDetailsList?.[reportItem.managerID] : emptyPersonalDetails,
@@ -372,7 +368,7 @@ function getReportSections(data: OnyxTypes.SearchResults['data'], metadata: Onyx
 
             const transaction = {
                 ...transactionItem,
-                action: getAction(data, key, currentUserAccountID),
+                action: getAction(data, key),
                 from,
                 to,
                 formattedFrom,

From a49494b773d953bbf7c026488570288909bb32a5 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 1 Nov 2024 15:08:45 -0600
Subject: [PATCH 40/49] rm violations code because its not ready

---
 src/libs/TransactionUtils/index.ts | 17 ++++++-----------
 src/libs/actions/IOU.ts            | 24 ------------------------
 src/types/onyx/SearchResults.ts    |  4 +---
 3 files changed, 7 insertions(+), 38 deletions(-)

diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts
index f8f9f9e9dc7e..faeebdf7e426 100644
--- a/src/libs/TransactionUtils/index.ts
+++ b/src/libs/TransactionUtils/index.ts
@@ -648,8 +648,8 @@ function hasPendingRTERViolation(transactionViolations?: TransactionViolations |
 /**
  * Check if there is broken connection violation.
  */
-function hasBrokenConnectionViolation(transactionID: string, allViolations?: OnyxCollection<TransactionViolations>): boolean {
-    const violations = getTransactionViolations(transactionID, allViolations ?? allTransactionViolations);
+function hasBrokenConnectionViolation(transactionID: string): boolean {
+    const violations = getTransactionViolations(transactionID, allTransactionViolations);
     return !!violations?.find(
         (violation) =>
             violation.name === CONST.VIOLATIONS.RTER &&
@@ -660,14 +660,9 @@ function hasBrokenConnectionViolation(transactionID: string, allViolations?: Ony
 /**
  * Check if user should see broken connection violation warning.
  */
-function shouldShowBrokenConnectionViolation(
-    transactionID: string,
-    report: OnyxEntry<Report> | SearchReport,
-    policy: OnyxEntry<Policy> | SearchPolicy,
-    allViolations?: OnyxCollection<TransactionViolations>,
-): boolean {
+function shouldShowBrokenConnectionViolation(transactionID: string, report: OnyxEntry<Report> | SearchReport, policy: OnyxEntry<Policy> | SearchPolicy): boolean {
     return (
-        hasBrokenConnectionViolation(transactionID, allViolations) &&
+        hasBrokenConnectionViolation(transactionID) &&
         (!PolicyUtils.isPolicyAdmin(policy) || ReportUtils.isOpenExpenseReport(report) || (ReportUtils.isProcessingReport(report) && PolicyUtils.isInstantSubmitEnabled(policy)))
     );
 }
@@ -675,9 +670,9 @@ function shouldShowBrokenConnectionViolation(
 /**
  * Check if there is pending rter violation in all transactionViolations with given transactionIDs.
  */
-function allHavePendingRTERViolation(transactionIds: string[], searchTransactionViolations?: OnyxCollection<TransactionViolations>): boolean {
+function allHavePendingRTERViolation(transactionIds: string[]): boolean {
     const transactionsWithRTERViolations = transactionIds.map((transactionId) => {
-        const transactionViolations = getTransactionViolations(transactionId, searchTransactionViolations ?? allTransactionViolations);
+        const transactionViolations = getTransactionViolations(transactionId, allTransactionViolations);
         return hasPendingRTERViolation(transactionViolations);
     });
     return transactionsWithRTERViolations.length > 0 && transactionsWithRTERViolations.every((value) => value === true);
diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts
index d3b54cb4e1aa..bb6c1a94c88b 100644
--- a/src/libs/actions/IOU.ts
+++ b/src/libs/actions/IOU.ts
@@ -7111,29 +7111,6 @@ function canIOUBePaid(
     );
 }
 
-function canReportBeSubmitted(
-    moneyRequestReport: OnyxEntry<OnyxTypes.Report> | SearchReport,
-    policy: OnyxEntry<OnyxTypes.Policy> | SearchPolicy,
-    currentUserAccountID: number,
-    transaction?: OnyxTypes.Transaction | SearchTransaction,
-    violations?: OnyxCollection<OnyxTypes.TransactionViolations>,
-) {
-    const isDraft = ReportUtils.isOpenExpenseReport(moneyRequestReport);
-    const {reimbursableSpend} = ReportUtils.getMoneyRequestSpendBreakdown(moneyRequestReport);
-    const hasAllPendingRTERViolations = TransactionUtils.allHavePendingRTERViolation([transaction?.transactionID ?? '-1'], violations);
-    const shouldShowBrokenConnectionViolation = TransactionUtils.shouldShowBrokenConnectionViolation(transaction?.transactionID ?? '-1', moneyRequestReport, policy, violations);
-    const isAdmin = policy?.role === CONST.POLICY.ROLE.ADMIN;
-
-    return (
-        !!moneyRequestReport &&
-        isDraft &&
-        reimbursableSpend !== 0 &&
-        !hasAllPendingRTERViolations &&
-        !shouldShowBrokenConnectionViolation &&
-        (moneyRequestReport?.ownerAccountID === currentUserAccountID || isAdmin || moneyRequestReport?.managerID === currentUserAccountID)
-    );
-}
-
 function getIOUReportActionToApproveOrPay(chatReport: OnyxEntry<OnyxTypes.Report>, excludedIOUReportID: string): OnyxEntry<ReportAction> {
     const chatReportActions = allReportActions?.[`${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${chatReport?.reportID}`] ?? {};
 
@@ -8645,6 +8622,5 @@ export {
     updateLastLocationPermissionPrompt,
     resolveDuplicates,
     getIOUReportActionToApproveOrPay,
-    canReportBeSubmitted,
 };
 export type {GPSPoint as GpsPoint, IOURequestType};
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index 26d258a0142a..af9b2c06265f 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -9,7 +9,6 @@ import type ONYXKEYS from '@src/ONYXKEYS';
 import type {InvoiceReceiver} from './Report';
 import type ReportActionName from './ReportActionName';
 import type ReportNameValuePairs from './ReportNameValuePairs';
-import type TransactionViolations from './TransactionViolation';
 
 /** Types of search data */
 type SearchDataTypes = ValueOf<typeof CONST.SEARCH.DATA_TYPES>;
@@ -354,8 +353,7 @@ type SearchResults = {
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_ACTIONS, Record<string, SearchReportAction>> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT, SearchReport> &
         PrefixedRecord<typeof ONYXKEYS.COLLECTION.POLICY, SearchPolicy> &
-        PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_NAME_VALUE_PAIRS, ReportNameValuePairs> &
-        PrefixedRecord<typeof ONYXKEYS.COLLECTION.TRANSACTION_VIOLATIONS, TransactionViolations>;
+        PrefixedRecord<typeof ONYXKEYS.COLLECTION.REPORT_NAME_VALUE_PAIRS, ReportNameValuePairs>;
 
     /** Whether search data is being fetched from server */
     isLoading?: boolean;

From e5ebd97fc43d87a8692cd8c2f6214a7d73e2b8b5 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Mon, 4 Nov 2024 14:48:46 -0700
Subject: [PATCH 41/49] fix selected color

---
 src/components/SelectionList/Search/ActionCell.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/SelectionList/Search/ActionCell.tsx b/src/components/SelectionList/Search/ActionCell.tsx
index d922f0a488d2..bf709c172d48 100644
--- a/src/components/SelectionList/Search/ActionCell.tsx
+++ b/src/components/SelectionList/Search/ActionCell.tsx
@@ -64,9 +64,8 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
         );
     }
 
-    const buttonInnerStyles = isSelected ? styles.buttonDefaultHovered : {};
-
     if (action === CONST.SEARCH.ACTION_TYPES.VIEW || shouldUseViewAction) {
+        const buttonInnerStyles = isSelected ? styles.buttonDefaultHovered : {};
         return isLargeScreenWidth ? (
             <Button
                 text={translate(actionTranslationsMap[CONST.SEARCH.ACTION_TYPES.VIEW])}
@@ -80,6 +79,7 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
         ) : null;
     }
 
+    const buttonInnerStyles = isSelected ? styles.buttonSuccessHovered : {};
     return (
         <Button
             text={text}

From ca22276d9dc0647a59f2e803d0e5936c43c4d036 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Tue, 5 Nov 2024 15:42:52 -0700
Subject: [PATCH 42/49] update code to handle array in pay action

---
 src/components/Search/types.ts                         |  8 ++++++++
 .../API/parameters/PayMoneyRequestOnSearchParams.ts    |  5 ++---
 src/libs/actions/Search.ts                             | 10 +++++-----
 3 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/src/components/Search/types.ts b/src/components/Search/types.ts
index 2fb034131c86..d496d05acec2 100644
--- a/src/components/Search/types.ts
+++ b/src/components/Search/types.ts
@@ -24,6 +24,13 @@ type SelectedTransactionInfo = {
 /** Model of selected results */
 type SelectedTransactions = Record<string, SelectedTransactionInfo>;
 
+/** Model of payment data used by Search bulk actions */
+type PaymentData = {
+    reportID: string;
+    amount: number;
+    paymentType: ValueOf<typeof CONST.IOU.PAYMENT_TYPE>;
+};
+
 type SortOrder = ValueOf<typeof CONST.SEARCH.SORT_ORDER>;
 type SearchColumnType = ValueOf<typeof CONST.SEARCH.TABLE_COLUMNS>;
 type ExpenseSearchStatus = ValueOf<typeof CONST.SEARCH.STATUS.EXPENSE>;
@@ -114,4 +121,5 @@ export type {
     ChatSearchStatus,
     SearchAutocompleteResult,
     AutocompleteRange,
+    PaymentData,
 };
diff --git a/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts b/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts
index 10174f0baa37..ee51a1961448 100644
--- a/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts
+++ b/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts
@@ -1,11 +1,10 @@
 type PayMoneyRequestOnSearchParams = {
     hash: number;
-    paymentType: string;
     /**
      * Stringified JSON object with type of following structure:
-     * Array<{reportID: string, amount: number}>
+     * Array<{reportID: string, amount: number, paymentMethod: string}>
      */
-    reportsAndAmounts: string;
+    paymentData: string;
 };
 
 export default PayMoneyRequestOnSearchParams;
diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index 556fe76df4d1..f343d0d6b849 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -2,7 +2,7 @@ import Onyx from 'react-native-onyx';
 import type {OnyxEntry, OnyxUpdate} from 'react-native-onyx';
 import type {ValueOf} from 'type-fest';
 import type {FormOnyxValues} from '@components/Form/types';
-import type {SearchQueryJSON} from '@components/Search/types';
+import type {PaymentData, SearchQueryJSON} from '@components/Search/types';
 import type {ReportListItemType, TransactionListItemType} from '@components/SelectionList/types';
 import * as API from '@libs/API';
 import type {ExportSearchItemsToCSVParams} from '@libs/API/parameters';
@@ -35,11 +35,11 @@ Onyx.connect({
 });
 
 function handleActionButtonPress(hash: number, item: TransactionListItemType | ReportListItemType, goToItem: () => void) {
-    const lastPolicyPaymentMethod = item.policyID ? lastPaymentMethod?.[item.policyID] : null;
+    const lastPolicyPaymentMethod = item.policyID ? (lastPaymentMethod?.[item.policyID] as ValueOf<typeof CONST.IOU.PAYMENT_TYPE>) : null;
     const amount = isReportListItemType(item) ? item.total ?? 0 : item.formattedTotal;
     switch (item.action) {
         case CONST.SEARCH.ACTION_TYPES.PAY:
-            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, lastPolicyPaymentMethod, {[item.reportID]: amount}) : goToItem();
+            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, [{reportID: item.reportID, amount, paymentType: lastPolicyPaymentMethod}]) : goToItem();
         case CONST.SEARCH.ACTION_TYPES.APPROVE:
             return approveMoneyRequestOnSearch(hash, [item.reportID]);
         default:
@@ -194,10 +194,10 @@ function approveMoneyRequestOnSearch(hash: number, reportIDList: string[]) {
     API.write(WRITE_COMMANDS.APPROVE_MONEY_REQUEST_ON_SEARCH, {hash, reportIDList}, {optimisticData, finallyData});
 }
 
-function payMoneyRequestOnSearch(hash: number, paymentType: string, reportsAndAmounts: Record<string, number>) {
+function payMoneyRequestOnSearch(hash: number, paymentData: PaymentData[]) {
     const {optimisticData, finallyData} = getOnyxLoadingData(hash);
 
-    API.write(WRITE_COMMANDS.PAY_MONEY_REQUEST_ON_SEARCH, {hash, paymentType, reportsAndAmounts: JSON.stringify(reportsAndAmounts)}, {optimisticData, finallyData});
+    API.write(WRITE_COMMANDS.PAY_MONEY_REQUEST_ON_SEARCH, {hash, paymentData: JSON.stringify(paymentData)}, {optimisticData, finallyData});
 }
 
 function unholdMoneyRequestOnSearch(hash: number, transactionIDList: string[]) {

From 148c72b7106f15c834c004966c47b8ccb5767a63 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Tue, 5 Nov 2024 15:46:30 -0700
Subject: [PATCH 43/49] update docs

---
 src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts b/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts
index ee51a1961448..a12c0fdc32f2 100644
--- a/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts
+++ b/src/libs/API/parameters/PayMoneyRequestOnSearchParams.ts
@@ -2,7 +2,7 @@ type PayMoneyRequestOnSearchParams = {
     hash: number;
     /**
      * Stringified JSON object with type of following structure:
-     * Array<{reportID: string, amount: number, paymentMethod: string}>
+     * Array<{reportID: string, amount: number, paymentType: string}>
      */
     paymentData: string;
 };

From 8d2222f1303a2b1d5a302c6d6a99d880373667a2 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 8 Nov 2024 12:37:13 -0700
Subject: [PATCH 44/49] add loading indicator

---
 .../SelectionList/Search/ActionCell.tsx           | 12 +++++++++++-
 .../Search/ExpenseItemHeaderNarrow.tsx            |  3 +++
 .../SelectionList/Search/ReportListItem.tsx       |  1 +
 .../SelectionList/Search/TransactionListItem.tsx  |  1 +
 .../Search/TransactionListItemRow.tsx             |  3 +++
 src/libs/actions/Search.ts                        | 15 +++++++++++++--
 src/types/onyx/SearchResults.ts                   |  3 +++
 7 files changed, 35 insertions(+), 3 deletions(-)

diff --git a/src/components/SelectionList/Search/ActionCell.tsx b/src/components/SelectionList/Search/ActionCell.tsx
index bf709c172d48..55e2cf6f849d 100644
--- a/src/components/SelectionList/Search/ActionCell.tsx
+++ b/src/components/SelectionList/Search/ActionCell.tsx
@@ -29,9 +29,18 @@ type ActionCellProps = {
     goToItem: () => void;
     isChildListItem?: boolean;
     parentAction?: string;
+    isLoading?: boolean;
 };
 
-function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth = true, isSelected = false, goToItem, isChildListItem = false, parentAction = ''}: ActionCellProps) {
+function ActionCell({
+    action = CONST.SEARCH.ACTION_TYPES.VIEW,
+    isLargeScreenWidth = true,
+    isSelected = false,
+    goToItem,
+    isChildListItem = false,
+    parentAction = '',
+    isLoading = false,
+}: ActionCellProps) {
     const {translate} = useLocalize();
     const theme = useTheme();
     const styles = useThemeStyles();
@@ -87,6 +96,7 @@ function ActionCell({action = CONST.SEARCH.ACTION_TYPES.VIEW, isLargeScreenWidth
             small
             style={[styles.w100]}
             innerStyles={buttonInnerStyles}
+            isLoading={isLoading}
             success
         />
     );
diff --git a/src/components/SelectionList/Search/ExpenseItemHeaderNarrow.tsx b/src/components/SelectionList/Search/ExpenseItemHeaderNarrow.tsx
index 185b9991e0a6..384262a78b15 100644
--- a/src/components/SelectionList/Search/ExpenseItemHeaderNarrow.tsx
+++ b/src/components/SelectionList/Search/ExpenseItemHeaderNarrow.tsx
@@ -28,6 +28,7 @@ type ExpenseItemHeaderNarrowProps = {
     isDisabled?: boolean | null;
     isDisabledCheckbox?: boolean;
     handleCheckboxPress?: () => void;
+    isLoading?: boolean;
 };
 
 function ExpenseItemHeaderNarrow({
@@ -44,6 +45,7 @@ function ExpenseItemHeaderNarrow({
     isDisabled,
     handleCheckboxPress,
     text,
+    isLoading = false,
 }: ExpenseItemHeaderNarrowProps) {
     const styles = useThemeStyles();
     const StyleUtils = useStyleUtils();
@@ -102,6 +104,7 @@ function ExpenseItemHeaderNarrow({
                     goToItem={onButtonPress}
                     isLargeScreenWidth={false}
                     isSelected={isSelected}
+                    isLoading={isLoading}
                 />
             </View>
         </View>
diff --git a/src/components/SelectionList/Search/ReportListItem.tsx b/src/components/SelectionList/Search/ReportListItem.tsx
index bdaeb915d943..4474ba7676aa 100644
--- a/src/components/SelectionList/Search/ReportListItem.tsx
+++ b/src/components/SelectionList/Search/ReportListItem.tsx
@@ -202,6 +202,7 @@ function ReportListItem<TItem extends ListItem>({
                                 action={reportItem.action}
                                 goToItem={handleOnButtonPress}
                                 isSelected={item.isSelected}
+                                isLoading={reportItem.isActionLoading}
                             />
                         </View>
                     )}
diff --git a/src/components/SelectionList/Search/TransactionListItem.tsx b/src/components/SelectionList/Search/TransactionListItem.tsx
index db97985a1612..5d52c243e44d 100644
--- a/src/components/SelectionList/Search/TransactionListItem.tsx
+++ b/src/components/SelectionList/Search/TransactionListItem.tsx
@@ -85,6 +85,7 @@ function TransactionListItem<TItem extends ListItem>({
                 canSelectMultiple={!!canSelectMultiple}
                 isButtonSelected={item.isSelected}
                 shouldShowTransactionCheckbox={false}
+                isLoading={false}
             />
         </BaseListItem>
     );
diff --git a/src/components/SelectionList/Search/TransactionListItemRow.tsx b/src/components/SelectionList/Search/TransactionListItemRow.tsx
index 37b0429b7caa..99ad5436aab2 100644
--- a/src/components/SelectionList/Search/TransactionListItemRow.tsx
+++ b/src/components/SelectionList/Search/TransactionListItemRow.tsx
@@ -59,6 +59,7 @@ type TransactionListItemRowProps = {
     isButtonSelected?: boolean;
     parentAction?: string;
     shouldShowTransactionCheckbox?: boolean;
+    isLoading?: boolean;
 };
 
 const getTypeIcon = (type?: SearchTransactionType) => {
@@ -257,6 +258,7 @@ function TransactionListItemRow({
     isButtonSelected = false,
     parentAction = '',
     shouldShowTransactionCheckbox,
+    isLoading = false,
 }: TransactionListItemRowProps) {
     const styles = useThemeStyles();
     const {isLargeScreenWidth} = useResponsiveLayout();
@@ -445,6 +447,7 @@ function TransactionListItemRow({
                         isChildListItem={isChildListItem}
                         parentAction={parentAction}
                         goToItem={onButtonPress}
+                        isLoading={isLoading}
                     />
                 </View>
             </View>
diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index f343d0d6b849..c7bb384c5a11 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -15,7 +15,7 @@ import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import FILTER_KEYS from '@src/types/form/SearchAdvancedFiltersForm';
 import type {LastPaymentMethod} from '@src/types/onyx';
-import type {SearchTransaction} from '@src/types/onyx/SearchResults';
+import type {SearchReport, SearchTransaction} from '@src/types/onyx/SearchResults';
 import * as Report from './Report';
 
 let currentUserEmail: string;
@@ -189,7 +189,18 @@ function holdMoneyRequestOnSearch(hash: number, transactionIDList: string[], com
 }
 
 function approveMoneyRequestOnSearch(hash: number, reportIDList: string[]) {
-    const {optimisticData, finallyData} = getOnyxLoadingData(hash);
+    const createOnyxLoadingData = (isLoading: boolean): OnyxUpdate[] => [
+        {
+            onyxMethod: Onyx.METHOD.MERGE,
+            key: `${ONYXKEYS.COLLECTION.SNAPSHOT}${hash}`,
+            value: {
+                data: Object.fromEntries(reportIDList.map((reportID) => [`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, {isActionLoading: isLoading}])) as Partial<SearchReport>,
+            },
+        },
+    ];
+
+    const optimisticData: OnyxUpdate[] = createOnyxLoadingData(true);
+    const finallyData: OnyxUpdate[] = createOnyxLoadingData(false);
 
     API.write(WRITE_COMMANDS.APPROVE_MONEY_REQUEST_ON_SEARCH, {hash, reportIDList}, {optimisticData, finallyData});
 }
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index af9b2c06265f..513aef080f93 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -146,6 +146,9 @@ type SearchReport = {
     /** Whether the report is archived */
     // eslint-disable-next-line @typescript-eslint/naming-convention
     private_isArchived?: string;
+
+    /** Whether the action is loading */
+    isActionLoading?: boolean;
 };
 
 /** Model of report action search result */

From c46c57111fc365abc678b26cd1bbb646c6dcc8af Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 8 Nov 2024 12:38:10 -0700
Subject: [PATCH 45/49] pass more props

---
 src/components/SelectionList/Search/ReportListItem.tsx         | 1 +
 src/components/SelectionList/Search/TransactionListItemRow.tsx | 1 +
 2 files changed, 2 insertions(+)

diff --git a/src/components/SelectionList/Search/ReportListItem.tsx b/src/components/SelectionList/Search/ReportListItem.tsx
index 4474ba7676aa..b03932db2532 100644
--- a/src/components/SelectionList/Search/ReportListItem.tsx
+++ b/src/components/SelectionList/Search/ReportListItem.tsx
@@ -168,6 +168,7 @@ function ReportListItem<TItem extends ListItem>({
                         action={reportItem.action}
                         onButtonPress={handleOnButtonPress}
                         containerStyle={[styles.ph3, styles.pt1half, styles.mb1half]}
+                        isLoading={reportItem.isActionLoading}
                     />
                 )}
                 <View style={[styles.flex1, styles.flexRow, styles.alignItemsCenter, styles.gap3, styles.ph3, styles.pv1half]}>
diff --git a/src/components/SelectionList/Search/TransactionListItemRow.tsx b/src/components/SelectionList/Search/TransactionListItemRow.tsx
index 99ad5436aab2..6bd071fa5ded 100644
--- a/src/components/SelectionList/Search/TransactionListItemRow.tsx
+++ b/src/components/SelectionList/Search/TransactionListItemRow.tsx
@@ -282,6 +282,7 @@ function TransactionListItemRow({
                         isDisabled={item.isDisabled}
                         isDisabledCheckbox={item.isDisabledCheckbox}
                         handleCheckboxPress={onCheckboxPress}
+                        isLoading={isLoading}
                     />
                 )}
 

From e5b5ed2571e1d754f083c6107e9f84915a82f562 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 8 Nov 2024 12:59:03 -0700
Subject: [PATCH 46/49] update callback to handle single transaction reports

---
 .../Search/TransactionListItem.tsx            |  2 +-
 src/libs/actions/Search.ts                    | 24 +++++++++++--------
 src/types/onyx/SearchResults.ts               |  3 +++
 3 files changed, 18 insertions(+), 11 deletions(-)

diff --git a/src/components/SelectionList/Search/TransactionListItem.tsx b/src/components/SelectionList/Search/TransactionListItem.tsx
index 5d52c243e44d..a23a7048644e 100644
--- a/src/components/SelectionList/Search/TransactionListItem.tsx
+++ b/src/components/SelectionList/Search/TransactionListItem.tsx
@@ -85,7 +85,7 @@ function TransactionListItem<TItem extends ListItem>({
                 canSelectMultiple={!!canSelectMultiple}
                 isButtonSelected={item.isSelected}
                 shouldShowTransactionCheckbox={false}
-                isLoading={false}
+                isLoading={transactionItem.isActionLoading}
             />
         </BaseListItem>
     );
diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index c7bb384c5a11..bb671fe7720d 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -10,7 +10,7 @@ import {WRITE_COMMANDS} from '@libs/API/types';
 import * as ApiUtils from '@libs/ApiUtils';
 import fileDownload from '@libs/fileDownload';
 import enhanceParameters from '@libs/Network/enhanceParameters';
-import {isReportListItemType} from '@libs/SearchUIUtils';
+import {isReportListItemType, isTransactionListItemType} from '@libs/SearchUIUtils';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
 import FILTER_KEYS from '@src/types/form/SearchAdvancedFiltersForm';
@@ -35,13 +35,15 @@ Onyx.connect({
 });
 
 function handleActionButtonPress(hash: number, item: TransactionListItemType | ReportListItemType, goToItem: () => void) {
-    const lastPolicyPaymentMethod = item.policyID ? (lastPaymentMethod?.[item.policyID] as ValueOf<typeof CONST.IOU.PAYMENT_TYPE>) : null;
-    const amount = isReportListItemType(item) ? item.total ?? 0 : item.formattedTotal;
+    const transactionID = isTransactionListItemType(item) ? item.transactionID : undefined;
     switch (item.action) {
-        case CONST.SEARCH.ACTION_TYPES.PAY:
+        case CONST.SEARCH.ACTION_TYPES.PAY: {
+            const lastPolicyPaymentMethod = item.policyID ? (lastPaymentMethod?.[item.policyID] as ValueOf<typeof CONST.IOU.PAYMENT_TYPE>) : null;
+            const amount = isReportListItemType(item) ? item.total ?? 0 : item.formattedTotal;
             return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, [{reportID: item.reportID, amount, paymentType: lastPolicyPaymentMethod}]) : goToItem();
+        }
         case CONST.SEARCH.ACTION_TYPES.APPROVE:
-            return approveMoneyRequestOnSearch(hash, [item.reportID]);
+            return approveMoneyRequestOnSearch(hash, [item.reportID], transactionID);
         default:
             return goToItem();
     }
@@ -188,19 +190,21 @@ function holdMoneyRequestOnSearch(hash: number, transactionIDList: string[], com
     API.write(WRITE_COMMANDS.HOLD_MONEY_REQUEST_ON_SEARCH, {hash, transactionIDList, comment}, {optimisticData, finallyData});
 }
 
-function approveMoneyRequestOnSearch(hash: number, reportIDList: string[]) {
-    const createOnyxLoadingData = (isLoading: boolean): OnyxUpdate[] => [
+function approveMoneyRequestOnSearch(hash: number, reportIDList: string[], transactionID?: string) {
+    const createActionLoadingData = (isLoading: boolean): OnyxUpdate[] => [
         {
             onyxMethod: Onyx.METHOD.MERGE,
             key: `${ONYXKEYS.COLLECTION.SNAPSHOT}${hash}`,
             value: {
-                data: Object.fromEntries(reportIDList.map((reportID) => [`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, {isActionLoading: isLoading}])) as Partial<SearchReport>,
+                data: transactionID
+                    ? {[`${ONYXKEYS.COLLECTION.TRANSACTION}${transactionID}`]: {isActionLoading: isLoading}}
+                    : (Object.fromEntries(reportIDList.map((reportID) => [`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, {isActionLoading: isLoading}])) as Partial<SearchReport>),
             },
         },
     ];
 
-    const optimisticData: OnyxUpdate[] = createOnyxLoadingData(true);
-    const finallyData: OnyxUpdate[] = createOnyxLoadingData(false);
+    const optimisticData: OnyxUpdate[] = createActionLoadingData(true);
+    const finallyData: OnyxUpdate[] = createActionLoadingData(false);
 
     API.write(WRITE_COMMANDS.APPROVE_MONEY_REQUEST_ON_SEARCH, {hash, reportIDList}, {optimisticData, finallyData});
 }
diff --git a/src/types/onyx/SearchResults.ts b/src/types/onyx/SearchResults.ts
index 513aef080f93..56ae9cc62379 100644
--- a/src/types/onyx/SearchResults.ts
+++ b/src/types/onyx/SearchResults.ts
@@ -333,6 +333,9 @@ type SearchTransaction = {
 
     /** Whether the transaction report has only a single transaction */
     isFromOneTransactionReport?: boolean;
+
+    /** Whether the action is loading */
+    isActionLoading?: boolean;
 };
 
 /** Types of searchable transactions */

From 66acf61a496c23428176c8f0fda8d6b3b93783b5 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 8 Nov 2024 13:22:46 -0700
Subject: [PATCH 47/49] add loading for pay action

---
 src/libs/actions/Search.ts | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index bb671fe7720d..a6aee93f8ac3 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -40,7 +40,7 @@ function handleActionButtonPress(hash: number, item: TransactionListItemType | R
         case CONST.SEARCH.ACTION_TYPES.PAY: {
             const lastPolicyPaymentMethod = item.policyID ? (lastPaymentMethod?.[item.policyID] as ValueOf<typeof CONST.IOU.PAYMENT_TYPE>) : null;
             const amount = isReportListItemType(item) ? item.total ?? 0 : item.formattedTotal;
-            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, [{reportID: item.reportID, amount, paymentType: lastPolicyPaymentMethod}]) : goToItem();
+            return lastPolicyPaymentMethod ? payMoneyRequestOnSearch(hash, [{reportID: item.reportID, amount, paymentType: lastPolicyPaymentMethod}], transactionID) : goToItem();
         }
         case CONST.SEARCH.ACTION_TYPES.APPROVE:
             return approveMoneyRequestOnSearch(hash, [item.reportID], transactionID);
@@ -209,8 +209,21 @@ function approveMoneyRequestOnSearch(hash: number, reportIDList: string[], trans
     API.write(WRITE_COMMANDS.APPROVE_MONEY_REQUEST_ON_SEARCH, {hash, reportIDList}, {optimisticData, finallyData});
 }
 
-function payMoneyRequestOnSearch(hash: number, paymentData: PaymentData[]) {
-    const {optimisticData, finallyData} = getOnyxLoadingData(hash);
+function payMoneyRequestOnSearch(hash: number, paymentData: PaymentData[], transactionID?: string) {
+    const createActionLoadingData = (isLoading: boolean): OnyxUpdate[] => [
+        {
+            onyxMethod: Onyx.METHOD.MERGE,
+            key: `${ONYXKEYS.COLLECTION.SNAPSHOT}${hash}`,
+            value: {
+                data: transactionID
+                    ? {[`${ONYXKEYS.COLLECTION.TRANSACTION}${transactionID}`]: {isActionLoading: isLoading}}
+                    : (Object.fromEntries(paymentData.map((item) => [`${ONYXKEYS.COLLECTION.REPORT}${item.reportID}`, {isActionLoading: isLoading}])) as Partial<SearchReport>),
+            },
+        },
+    ];
+
+    const optimisticData: OnyxUpdate[] = createActionLoadingData(true);
+    const finallyData: OnyxUpdate[] = createActionLoadingData(false);
 
     API.write(WRITE_COMMANDS.PAY_MONEY_REQUEST_ON_SEARCH, {hash, paymentData: JSON.stringify(paymentData)}, {optimisticData, finallyData});
 }

From 7c2034205afdbdf6b296293d0bcc5bf2db455aed Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 8 Nov 2024 13:26:42 -0700
Subject: [PATCH 48/49] add comment

---
 src/libs/actions/Search.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/libs/actions/Search.ts b/src/libs/actions/Search.ts
index a6aee93f8ac3..ecd0fc26785a 100644
--- a/src/libs/actions/Search.ts
+++ b/src/libs/actions/Search.ts
@@ -35,7 +35,10 @@ Onyx.connect({
 });
 
 function handleActionButtonPress(hash: number, item: TransactionListItemType | ReportListItemType, goToItem: () => void) {
+    // The transactionID is needed to handle actions taken on `status:all` where transactions on single expense reports can be approved/paid.
+    // We need the transactionID to display the loading indicator for that list item's action.
     const transactionID = isTransactionListItemType(item) ? item.transactionID : undefined;
+
     switch (item.action) {
         case CONST.SEARCH.ACTION_TYPES.PAY: {
             const lastPolicyPaymentMethod = item.policyID ? (lastPaymentMethod?.[item.policyID] as ValueOf<typeof CONST.IOU.PAYMENT_TYPE>) : null;

From ea0ae146a96d716c4980897d156a31f84c164777 Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Fri, 8 Nov 2024 13:35:42 -0700
Subject: [PATCH 49/49] delete empty file

---
 src/components/SelectionList/SearchTableHeaderColumn.tsx | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 delete mode 100644 src/components/SelectionList/SearchTableHeaderColumn.tsx

diff --git a/src/components/SelectionList/SearchTableHeaderColumn.tsx b/src/components/SelectionList/SearchTableHeaderColumn.tsx
deleted file mode 100644
index e69de29bb2d1..000000000000