Skip to content

Commit

Permalink
Merge pull request #18173 from Expensify/jasper-removeUnusedParticipa…
Browse files Browse the repository at this point in the history
…ntsFiltering

Remove unnecessary participants filtering
  • Loading branch information
jasperhuangg authored Apr 28, 2023
2 parents 420ea78 + 70b5bb4 commit bc051ba
Showing 1 changed file with 18 additions and 20 deletions.
38 changes: 18 additions & 20 deletions src/pages/ReportParticipantsPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,27 +57,25 @@ const defaultProps = {
const getAllParticipants = (report, personalDetails) => {
const {participants} = report;

return _.chain(participants)
.compact()
.map(participants, (login) => {
const userLogin = Str.removeSMSDomain(login);
const userPersonalDetail = lodashGet(personalDetails, login, {displayName: userLogin, avatar: ''});
return _.map(participants, (login) => {
const userLogin = Str.removeSMSDomain(login);
const userPersonalDetail = lodashGet(personalDetails, login, {displayName: userLogin, avatar: ''});

return ({
alternateText: userLogin,
displayName: userPersonalDetail.displayName,
icons: [{
source: ReportUtils.getAvatar(userPersonalDetail.avatar, login),
name: login,
type: CONST.ICON_TYPE_AVATAR,
}],
keyForList: userLogin,
login,
text: userPersonalDetail.displayName,
tooltipText: userLogin,
participantsList: [{login, displayName: userPersonalDetail.displayName}],
});
}).value();
return ({
alternateText: userLogin,
displayName: userPersonalDetail.displayName,
icons: [{
source: ReportUtils.getAvatar(userPersonalDetail.avatar, login),
name: login,
type: CONST.ICON_TYPE_AVATAR,
}],
keyForList: userLogin,
login,
text: userPersonalDetail.displayName,
tooltipText: userLogin,
participantsList: [{login, displayName: userPersonalDetail.displayName}],
});
});
};

const ReportParticipantsPage = (props) => {
Expand Down

0 comments on commit bc051ba

Please sign in to comment.