Skip to content

Commit

Permalink
Merge pull request #32330 from dukenv0307/fix/30296
Browse files Browse the repository at this point in the history
Fix new user does not appear in start chat after request IOU
  • Loading branch information
madmax330 authored Dec 18, 2023
2 parents 7ad6edc + 397a4f4 commit ca3e377
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/libs/OptionsListUtils.js
Original file line number Diff line number Diff line change
Expand Up @@ -1213,7 +1213,7 @@ function getOptions(
// This is a temporary fix for all the logic that's been breaking because of the new privacy changes
// See https://github.com/Expensify/Expensify/issues/293465 for more context
// Moreover, we should not override the personalDetails object, otherwise the createOption util won't work properly, it returns incorrect tooltipText
const havingLoginPersonalDetails = !includeP2P ? {} : _.pick(personalDetails, (detail) => Boolean(detail.login));
const havingLoginPersonalDetails = !includeP2P ? {} : _.pick(personalDetails, (detail) => Boolean(detail.login) && !detail.isOptimisticPersonalDetail);
let allPersonalDetailsOptions = _.map(havingLoginPersonalDetails, (personalDetail) =>
createOption([personalDetail.accountID], personalDetails, reportMapForAccountIDs[personalDetail.accountID], reportActions, {
showChatPreviewLine,
Expand Down Expand Up @@ -1391,7 +1391,7 @@ function getOptions(
}

return {
personalDetails: _.filter(personalDetailsOptions, (personalDetailsOption) => !personalDetailsOption.isOptimisticPersonalDetail),
personalDetails: personalDetailsOptions,
recentReports: recentReportOptions,
userToInvite: canInviteUser ? userToInvite : null,
currentUserOption,
Expand Down

0 comments on commit ca3e377

Please sign in to comment.