[$500] Desktop - Avatar isn't showing in the chat row until clicking into that row

[kbecciv]

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

---

Version Number: 1.3.80.1
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
Expensify/Expensify Issue URL:
Issue reported by: @kevinksullivan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1697055821976289

Action Performed:

1. Sent a DM message to ~20 customers via our internal compose tool
2. Logged in
3. Saw the DMs

Expected Result:

All DMs would have the avatar of the person I am speaking with in the row

Actual Result:

Some of the DMs didn't show an avatar until clicking into that DM

Workaround:

Unknown

Platforms:

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

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

http://g.recordit.co/96JvXGqDgK.gif

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~016c1d1ffb1e39b663
• Upwork Job ID: 1712488974749863936
• Last Price Increase: 2023-11-02

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~016c1d1ffb1e39b663

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @situchan (External)

[melvin-bot]

@stephanieelliott, @situchan Whoops! This issue is 2 days overdue. Let's get this updated quick!

[stephanieelliott]

Seeing as this applies to the Compose tool this will need to be internal.

[melvin-bot]

Current assignee @situchan is eligible for the Internal assigner, not assigning anyone new.

[stephanieelliott]

Not overdue, forgot I to also add Engineering label.

Just want to check first though -- @situchan do you agree that this needs an internal engineer to make BE changes?

[melvin-bot]

@stephanieelliott, @situchan Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[situchan]

@stephanieelliott I think we should fix this in frontend.

[stephanieelliott]

Ah ok, thanks @situchan, adjusting labels.

[barttom]

Could You just explain what is 'internal compose tool'?
Just wonder if I'm able to reproduce that by sending messages from the app
@situchan @stephanieelliott

[situchan]

I am not sure. @kevinksullivan or @stephanieelliott may have answer.

[stephanieelliott]

Hey @kevinksullivan are you referring to the Compose tool in Concierge? Or are you referring to sending a group DM in newdot?

[barttom]

@kevinksullivan please send more information on how You send these 20s messages. That helps me to reproduce the issue ;)

[stephanieelliott]

Looking at the thread in Slack, this seems to be our internal composer tool. As discussed there we should move this back to internal:

https://expensify.slack.com/archives/C049HHMV9SM/p1697055821976289

[melvin-bot]

Current assignee @situchan is eligible for the Internal assigner, not assigning anyone new.

[melvin-bot]

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

[puneetlath]

Does this affect the users receiving the messages? Or just our internal employees?

[stephanieelliott]

@puneetlath it affects the message sender only - so only our internal employees in this case!

[puneetlath]

Ok got it. Seems not super urgent then.

And when you sign out and sign back in, are the avatars in the LHN? My guess is that we don't send a pusher update which is why this is the case.

[melvin-bot]

@puneetlath @stephanieelliott @situchan this issue is now 4 weeks old and preventing us from maintaining WAQ. This should now be your highest priority. Please post below what your plan is to get a PR in review ASAP. Thanks!

[puneetlath]

@stephanieelliott just a friendly bump on the question above when you have a chance.

[stephanieelliott]

Just tested @puneetlath -- yep, the avatars populate in the LHN on signout/signin!

[stephanieelliott]

Hey @puneetlath what do you think about this one -- worth fixing? If yes, does it need to be fixed internally or is there a portion we can make external?

[puneetlath]

I'm thinking that it's not worth fixing right now since it only affects internal employees. What do you think?

[stephanieelliott]

I agree, esp considering it doesn't affect functionality. Let's close it for now!