Skip to content

Commit

Permalink
Merge pull request #17625 from Expensify/flo_preventCrash
Browse files Browse the repository at this point in the history
Prevent App crash when report participant personal details do not exist
  • Loading branch information
mountiny authored Apr 19, 2023
2 parents b6f6f6a + 0e64b41 commit 20fab46
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/libs/SidebarUtils.js
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,7 @@ function getOptionData(reportID) {
const hasMultipleParticipants = participantPersonalDetailList.length > 1 || result.isChatRoom || result.isPolicyExpenseChat;
const subtitle = ReportUtils.getChatRoomSubtitle(report, policies);

const login = Str.removeSMSDomain(personalDetail.login);
const login = Str.removeSMSDomain(personalDetail.login || '');
const formattedLogin = Str.isSMSLogin(login) ? LocalePhoneNumber.formatPhoneNumber(login) : login;

// We only create tooltips for the first 10 users or so since some reports have hundreds of users, causing performance to degrade.
Expand Down

0 comments on commit 20fab46

Please sign in to comment.