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

Chat - Previous avatar is displayed instead of New one, avatar is updated only after page refresh #19470

Closed
3 of 6 tasks
kbecciv opened this issue May 23, 2023 · 7 comments
Closed
3 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Needs Reproduction Reproducible steps needed

Comments

@kbecciv
Copy link

kbecciv commented May 23, 2023

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Issue found when executing PR #19107

Action Performed:

  1. Log in with 2 users that have chat history: userA and userB;
  2. From userA, click up on userB's name to open the Details Page;
  3. From userB, change your profile picture;
  4. From userA, click up on userB's profile picture.

Expected Result:

Window opens showing the new updated picture in the preview.

Actual Result:

Window opens showing previous avatar (Web)
Window opens with never-ending loading circle (mWeb and Android native)

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: 1.3.17.0

Reproducible in staging?: yes

Reproducible in production?: yes

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://platform.applause.com/services/links/v1/external/415e9099b5b7ac62f7b3048450fa08f07f821276c7cc36edc94088b5ffe67ed5

https://platform.applause.com/services/links/v1/external/bfc964ab693c5e0d4b73729d58312cee0e624f2df7dc26b1b77280eee9b34771

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels May 23, 2023
@melvin-bot
Copy link

melvin-bot bot commented May 23, 2023

Triggered auto assignment to @zanyrenney (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@melvin-bot
Copy link

melvin-bot bot commented May 23, 2023

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@DanutGavrus
Copy link
Contributor

PR 19107 addresses the case in which user's B profile picture is updated in user's A chat, but keeping the Details Page opened and later pressing on the new profile picture, was still showing the old one. I highly suggest watching these videos from the original issue for easier understanding.

In this video from 0:28 to 0:35 you can see that the old picture remains the same in the whole chat:
Screenshot from 2023-05-23 21-48-43
Same happens in this video after 1:06 when you navigate back to the chat, you have the old picture everywhere.

It looks like having some internet connection problems, or the mechanism which auto updates the chat does not work, which would be a different issue IMO.

I'm not able to reproduce this on staging though. Do you keep having this problem? Anyone else has this problem?

@DanutGavrus
Copy link
Contributor

@fedirjh What are your thoughts about this? Are you able to reproduce?

@zanyrenney
Copy link
Contributor

I can't reproduce this!

@zanyrenney zanyrenney added the Needs Reproduction Reproducible steps needed label May 24, 2023
@fedirjh
Copy link
Contributor

fedirjh commented May 24, 2023

I am unable to reproduce this issue

720.mov

@zanyrenney
Copy link
Contributor

Yeah i don't think this is still happening, I am going to close this out for now! Can you reopen if we uncover other use cases where this is still happening please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Needs Reproduction Reproducible steps needed
Projects
None yet
Development

No branches or pull requests

4 participants