Skip to content

Commit

Permalink
Merge pull request Expensify#47638 from BhuvaneshPatil/47282-member-c…
Browse files Browse the repository at this point in the history
…ount-issue-track-expense

fix: filter non number values
  • Loading branch information
cristipaval authored Aug 20, 2024
2 parents 0e4f02e + ed910fa commit 456b974
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/libs/ReportUtils.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import {format} from 'date-fns';
import {Str} from 'expensify-common';
import {isEmpty} from 'lodash';
import {isEmpty, isNumber} from 'lodash';
import lodashEscape from 'lodash/escape';
import lodashFindLastIndex from 'lodash/findLastIndex';
import lodashIntersection from 'lodash/intersection';
Expand Down Expand Up @@ -2083,7 +2083,7 @@ function getParticipantsAccountIDsForDisplay(report: OnyxEntry<Report>, shouldEx
});
}

return participantsEntries.map(([accountID]) => Number(accountID));
return participantsEntries.map(([accountID]) => Number(accountID)).filter((accountID) => isNumber(accountID));
}

function getParticipantsList(report: Report, personalDetails: OnyxEntry<PersonalDetailsList>, isRoomMembersList = false): number[] {
Expand Down

0 comments on commit 456b974

Please sign in to comment.