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

[HOLD for payment on July 7] Web - Profile - Deleted Profile name updated only after page refresh #3778

Closed
kavimuru opened this issue Jun 28, 2021 · 6 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@kavimuru
Copy link

kavimuru commented Jun 28, 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!

Upwork Job: https://www.upwork.com/jobs/~0133c0817f9a59ef1d


Action Performed:

  1. Go to https://staging.expensify.cash
  2. Log in with user (should Last and First name filled) and click on Avatar Picture
  3. Click on Profile
  4. Delete Last and First name

Expected Result:

Profile page should be displayed as email or phone number. it should work without the refresh

Actual Result:

Profile name appears until the page is refreshed

Workaround:

Unknown

Platform:

Where is this issue occurring?

Web ✔️
iOS ✔️
Android ✔️
Desktop App
Mobile Web ✔️

Version Number: 1.0.74-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Bug5128938_Recording__184.mp4

Expensify/Expensify Issue URL:

View all open jobs on Upwork

@MelvinBot
Copy link

Triggered auto assignment to @Jag96 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@Jag96 Jag96 removed their assignment Jun 29, 2021
@Jag96 Jag96 added the External Added to denote the issue can be worked on by a contributor label Jun 29, 2021
@MelvinBot
Copy link

Triggered auto assignment to @kevinksullivan (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@rdjuric
Copy link
Contributor

rdjuric commented Jun 29, 2021

Proposal

This happens because we're not merging our local changes in the case of an empty first and last name. This conditional looks unnecessary since our getDisplayName function already deals with the case of an empty name.

By removing it we get the expected behavior:

web.mov

@Jag96 Jag96 assigned Jag96 and unassigned kevinksullivan Jun 30, 2021
@Jag96 Jag96 added the Exported label Jun 30, 2021
@Jag96
Copy link
Contributor

Jag96 commented Jun 30, 2021

@rdjuric good find! Your proposal looks good to me, I've invited you to the Upwork job (https://www.upwork.com/jobs/~0133c0817f9a59ef1d). Please accept the invite there and once the offer is sent you're good to start on a PR!

@Jag96
Copy link
Contributor

Jag96 commented Jun 30, 2021

Offer sent!

@Jag96 Jag96 added Weekly KSv2 and removed Daily KSv2 labels Jun 30, 2021
@Jag96 Jag96 changed the title Web - Profile - Deleted Profile name updated only after page refresh [HOLD for payment on July 7] Web - Profile - Deleted Profile name updated only after page refresh Jul 1, 2021
@Jag96
Copy link
Contributor

Jag96 commented Jul 7, 2021

Paid!

@Jag96 Jag96 closed this as completed Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
None yet
Development

No branches or pull requests

5 participants