From 2bd1b7e8de150a7db403bfd14eee5f2d21fed35d Mon Sep 17 00:00:00 2001 From: ShridharGoel <35566748+ShridharGoel@users.noreply.github.com> Date: Thu, 18 Dec 2025 20:14:27 +0530 Subject: [PATCH] Filter out to-be-deleted accounts before showing list of members --- src/pages/ReportAddApproverPage.tsx | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/pages/ReportAddApproverPage.tsx b/src/pages/ReportAddApproverPage.tsx index c8d598082d98..dedcb8632079 100644 --- a/src/pages/ReportAddApproverPage.tsx +++ b/src/pages/ReportAddApproverPage.tsx @@ -60,11 +60,11 @@ function ReportAddApproverPage({report, isLoadingReportData, policy}: ReportAddA if (!email) { return null; } - const accountID = Number(policyMemberEmailsToAccountIDs[email] ?? CONST.DEFAULT_NUMBER_ID); - const isPendingDelete = employeeList?.[accountID]?.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE; + const accountID = policyMemberEmailsToAccountIDs[email]; + const isPendingDelete = employee?.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE; - // Filter the current report approver and members which are pending for deletion - if (report.managerID === accountID || isPendingDelete || !isAllowedToApproveExpenseReport(report, accountID, policy)) { + // Filter the current report approver, members which are pending for deletion, or members we cannot map to an account + if (!accountID || report.managerID === accountID || isPendingDelete || !isAllowedToApproveExpenseReport(report, accountID, policy)) { return null; }