Closed kavimuru closed 1 year ago
Triggered auto assignment to @jliexpensify (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)@expensify.sms domain being shown at various places
We're not checking if the user login is a phone number and displaying the login directly here and here.
We need to use the following here:
Str.isSMSLogin(personalDetail.login) ? LocalePhoneNumber.formatPhoneNumber(personalDetail.login) : personalDetail.login
Also, the following needs to be used here:
this.props.formatPhoneNumber(this.props.session.email)
None
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
App should not display @expensify.sms after contact number
Actual Result:
App displays @expensify.sms after contact number in share code and on tooltip when hovered over profiles in split bill message
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.19-2 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/07048f29-3c07-46e4-9a7f-84f82e69e975
https://github.com/Expensify/App/assets/43996225/95d659c9-8af0-4283-997c-76722da41930
Expensify/Expensify Issue URL: Issue reported by: @dhanashree-sawant Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1684849196517579
View all open jobs on GitHub