Skip to content

Commit

Permalink
Merge pull request #42036 from nkdengineer/fix/41832
Browse files Browse the repository at this point in the history
Fix member option appear in transaction thread report detail page
  • Loading branch information
rlinoz authored May 28, 2024
2 parents 322d6fe + 0843cb6 commit a1868c8
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions src/libs/ReportUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3624,8 +3624,8 @@ function buildOptimisticIOUReport(payeeAccountID: number, payerAccountID: number
const payerEmail = 'login' in personalDetails ? personalDetails.login : '';

const participants: Participants = {
[payeeAccountID]: {hidden: false},
[payerAccountID]: {hidden: false},
[payeeAccountID]: {hidden: true},
[payerAccountID]: {hidden: true},
};

return {
Expand Down Expand Up @@ -4338,10 +4338,11 @@ function buildOptimisticChatReport(
description = '',
avatarUrl = '',
optimisticReportID = '',
shouldShowParticipants = true,
): OptimisticChatReport {
const participants = participantList.reduce((reportParticipants: Participants, accountID: number) => {
const participant: ReportParticipant = {
hidden: false,
hidden: !shouldShowParticipants,
role: accountID === currentUserAccountID ? CONST.REPORT.ROLE.ADMIN : CONST.REPORT.ROLE.MEMBER,
};
// eslint-disable-next-line no-param-reassign
Expand Down Expand Up @@ -4890,6 +4891,8 @@ function buildTransactionThread(
moneyRequestReport?.reportID,
'',
'',
'',
false,
);
}

Expand Down

0 comments on commit a1868c8

Please sign in to comment.