Skip to content

Commit

Permalink
Merge pull request Expensify#32632 from bernhardoj/fix/31720-mention-…
Browse files Browse the repository at this point in the history
…tooltip

Fix tooltip doesn't show on a mention with a user we never chat before
  • Loading branch information
bondydaa authored Dec 13, 2023
2 parents 9f00f67 + a0b4d1c commit 2c3bc31
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ function BaseUserDetailsTooltip(props) {
const personalDetails = usePersonalDetails();

const userDetails = lodashGet(personalDetails, props.accountID, props.fallbackUserDetails);
let userDisplayName = ReportUtils.getDisplayNameForParticipant(props.accountID);
let userDisplayName = ReportUtils.getDisplayNameForParticipant(props.accountID) || (userDetails.displayName ? userDetails.displayName.trim() : '');
let userLogin = (userDetails.login || '').trim() && !_.isEqual(userDetails.login, userDetails.displayName) ? Str.removeSMSDomain(userDetails.login) : '';
let userAvatar = userDetails.avatar;
let userAccountID = props.accountID;
Expand Down

0 comments on commit 2c3bc31

Please sign in to comment.