Skip to content

Commit

Permalink
Merge pull request #42003 from Expensify/yuwen-removePronouns
Browse files Browse the repository at this point in the history
Remove Pronouns from report welcome text
  • Loading branch information
puneetlath authored May 10, 2024
2 parents e1ebfd0 + 1c80762 commit bf2ce72
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/components/ReportWelcomeText.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -145,9 +145,9 @@ function ReportWelcomeText({report, policy, personalDetails}: ReportWelcomeTextP
{isDefault && (
<Text>
<Text>{translate('reportActionsView.beginningOfChatHistory')}</Text>
{displayNamesWithTooltips.map(({displayName, pronouns, accountID}, index) => (
{displayNamesWithTooltips.map(({displayName, accountID}, index) => (
// eslint-disable-next-line react/no-array-index-key
<Text key={`${displayName}${pronouns}${index}`}>
<Text key={`${displayName}${index}`}>
<UserDetailsTooltip accountID={accountID}>
{ReportUtils.isOptimisticPersonalDetail(accountID) ? (
<Text style={[styles.textStrong]}>{displayName}</Text>
Expand All @@ -161,7 +161,6 @@ function ReportWelcomeText({report, policy, personalDetails}: ReportWelcomeTextP
</Text>
)}
</UserDetailsTooltip>
{!!pronouns && <Text>{` (${pronouns})`}</Text>}
{index === displayNamesWithTooltips.length - 1 && <Text>.</Text>}
{index === displayNamesWithTooltips.length - 2 && <Text>{` ${translate('common.and')} `}</Text>}
{index < displayNamesWithTooltips.length - 2 && <Text>, </Text>}
Expand Down

0 comments on commit bf2ce72

Please sign in to comment.