Skip to content

Commit 419aef0

Browse files
arosiclairOSBotify
authored andcommitted
Merge pull request #76041 from Expensify/revert-75410-fix/74900
Revert "Previous selection is cleared when selecting another member in Expenses From" (cherry picked from commit 2c9ad8d) (cherry-picked to staging by arosiclair)
1 parent dff21e5 commit 419aef0

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/components/ApproverSelectionList.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, {useMemo} from 'react';
1+
import React, {useMemo, useState} from 'react';
22
import type {SectionListData} from 'react-native';
33
import useDebouncedState from '@hooks/useDebouncedState';
44
import {useMemoizedLazyIllustrations} from '@hooks/useLazyAsset';
@@ -84,7 +84,7 @@ function ApproverSelectionList({
8484
const shouldShowTextInput = shouldShowTextInputProp ?? allApprovers?.length >= CONST.STANDARD_LIST_ITEM_LIMIT;
8585
const lazyIllustrations = useMemoizedLazyIllustrations(['TurtleInShell']);
8686

87-
const selectedMembers = useMemo(() => allApprovers.filter((approver) => approver.isSelected), [allApprovers]);
87+
const [selectedMembers, setSelectedMembers] = useState<SelectionListApprover[]>([]);
8888

8989
// eslint-disable-next-line rulesdir/no-negated-variables
9090
const shouldShowNotFoundView = (isEmptyObject(policy) && !isLoadingReportData) || !isPolicyAdmin(policy) || isPendingDeletePolicy(policy) || shouldShowNotFoundViewProp;
@@ -117,6 +117,7 @@ function ApproverSelectionList({
117117
? selectedMembers.filter((selectedOption) => selectedOption.login !== member.login)
118118
: [...selectedMembers, {...member, isSelected: true}];
119119
}
120+
setSelectedMembers(newSelectedApprovers);
120121
if (onSelectApprover) {
121122
onSelectApprover(newSelectedApprovers);
122123
}

0 commit comments

Comments
 (0)