Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

@Expensify.sms is visible in the tooltip when hovering over the user #3574

Closed
isagoico opened this issue Jun 14, 2021 · 1 comment · Fixed by #3580
Closed

@Expensify.sms is visible in the tooltip when hovering over the user #3574

isagoico opened this issue Jun 14, 2021 · 1 comment · Fixed by #3580
Assignees
Labels
Daily KSv2 Engineering Reviewing Has a PR in review

Comments

@isagoico
Copy link

isagoico commented Jun 14, 2021

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:

@expensify.sms should not be seen in any area of the app.

Expected Result:

@expensify.sms is visible when hovering over a user.

Actual Result:

  1. Log in to e.cash
  2. Navigate to a conversation with a user that has a main phone number log in,
  3. Hover over the name of the user.

Workaround:

No need, visual issue.

Platform:

Where is this issue occurring?

Web ✔️
iOS
Android
Desktop App ✔️
Mobile Web

Version Number: 1.0.68-0

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:
image

Expensify/Expensify Issue URL:

View all open jobs on Upwork


From @trjExpensify https://expensify.slack.com/archives/C01GTK53T8Q/p1623410626208600

Tooltip on hover includes the @expensify.sms portion for accounts with a phone number as a primary login (LHN, in the DM, and Search)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 Engineering Reviewing Has a PR in review
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants