Skip to content

Commit dc8e2eb

Browse files
authored
Merge pull request Expensify#89268 from callstack-internal/perf/approvals-expenses-from-page-freeze
2 parents 14a7d1b + b7eeaa8 commit dc8e2eb

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

src/pages/workspace/workflows/approvals/WorkspaceWorkflowsApprovalsExpensesFromPage.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ function WorkspaceWorkflowsApprovalsExpensesFromPage({policy, isLoadingReportDat
4646
const shouldShowListEmptyContent = !isLoadingApprovalWorkflow && approvalWorkflow?.availableMembers.length === 0;
4747
const firstApprover = approvalWorkflow?.originalApprovers?.[0]?.email ?? '';
4848
const isCreateAction = approvalWorkflow?.action === CONST.APPROVAL_WORKFLOW.ACTION.CREATE;
49+
const policyMemberEmailsToAccountIDs = getMemberAccountIDsForWorkspace(policy?.employeeList);
4950

5051
// Build a map of member emails to their existing workflow's approver email (for non-default workflows only)
5152
const membersInExistingWorkflows = (() => {
@@ -75,7 +76,6 @@ function WorkspaceWorkflowsApprovalsExpensesFromPage({policy, isLoadingReportDat
7576
// eslint-disable-next-line react-hooks/set-state-in-effect
7677
setSelectedMembers(
7778
approvalWorkflow.members.map((member) => {
78-
const policyMemberEmailsToAccountIDs = getMemberAccountIDsForWorkspace(policy?.employeeList);
7979
const accountID = Number(policyMemberEmailsToAccountIDs[member.email] ?? '');
8080

8181
return {
@@ -95,15 +95,14 @@ function WorkspaceWorkflowsApprovalsExpensesFromPage({policy, isLoadingReportDat
9595
};
9696
}),
9797
);
98-
}, [approvalWorkflow?.members, policy?.employeeList, policy?.owner, translate, icons.FallbackAvatar]);
98+
}, [approvalWorkflow?.members, policy?.employeeList, policy?.owner, policyMemberEmailsToAccountIDs, translate, icons.FallbackAvatar]);
9999

100100
const approversEmail = approvalWorkflow?.approvers.map((member) => member?.email);
101101
const allApprovers: SelectionListApprover[] = [...selectedMembers];
102102

103103
if (approvalWorkflow?.availableMembers) {
104104
const availableMembers = approvalWorkflow.availableMembers
105105
.map((member) => {
106-
const policyMemberEmailsToAccountIDs = getMemberAccountIDsForWorkspace(policy?.employeeList);
107106
const accountID = Number(policyMemberEmailsToAccountIDs[member.email] ?? '');
108107

109108
return {

0 commit comments

Comments
 (0)