Skip to content

Commit

Permalink
Merge pull request #35077 from lukemorawski/34658-money_request_no_re…
Browse files Browse the repository at this point in the history
…sults_found_fix

Money Request "no results found" header fix
  • Loading branch information
mountiny authored Feb 5, 2024
2 parents 6c2882a + 09b3429 commit 8e069a3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,7 @@ function MoneyTemporaryForRefactorRequestParticipantsSelector({
const headerMessage = useMemo(
() =>
OptionsListUtils.getHeaderMessage(
_.get(newChatOptions, 'personalDetails.length', 0) + _.get(newChatOptions, 'recentReports.length', 0) !== 0,
_.get(newChatOptions, 'personalDetails', []).length + _.get(newChatOptions, 'recentReports', []).length !== 0,
Boolean(newChatOptions.userToInvite),
searchTerm.trim(),
maxParticipantsReached,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -259,13 +259,13 @@ function MoneyRequestParticipantsSelector({
const headerMessage = useMemo(
() =>
OptionsListUtils.getHeaderMessage(
newChatOptions.personalDetails.length + newChatOptions.recentReports.length !== 0,
_.get(newChatOptions, 'personalDetails', []).length + _.get(newChatOptions, 'recentReports', []).length !== 0,
Boolean(newChatOptions.userToInvite),
searchTerm.trim(),
maxParticipantsReached,
_.some(participants, (participant) => participant.searchText.toLowerCase().includes(searchTerm.trim().toLowerCase())),
),
[maxParticipantsReached, newChatOptions.personalDetails.length, newChatOptions.recentReports.length, newChatOptions.userToInvite, participants, searchTerm],
[maxParticipantsReached, newChatOptions, participants, searchTerm],
);

// Right now you can't split a request with a workspace and other additional participants
Expand Down

0 comments on commit 8e069a3

Please sign in to comment.