Desktop - Chat - User name is not displayed when initiating chat for the first time

[kavimuru]

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

---

Action Performed:

1. Launch the app and login
2. Click the green fab button select New Chat
3. Search for a user who has expensify account but haven't chatted before.
4. Click on that user to initiate chat

Expected Result:

User name is displayed on the chat conversation page

Actual Result:

User name is not displayed on the chat conversation page

Workaround:

Unknown

Platform:

Where is this issue occurring?

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

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

Bug5152909_Recording__968.mp4

Expensify/Expensify Issue URL:

View all open jobs on Upwork

[OSBotify]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[MelvinBot]

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

[Jag96]

It looks like this is also related to https://github.com/Expensify/Auth/pull/5747, for new chats when we call API.GetReportSummaryList, the returned data is an empty set, which means we are never setting the participants data here.

I confirmed by checking the return value for two different reportIDs, one that has chat data in it and one that contains no data using the format https://www.expensify.com/api?command=Get&returnValueList=reportSummaryList&reportIDList=REPORT_ID_HERE

Since this is going to be an internal fix, assigning myself and making it internal.

[marcaaron]

Sorry / thanks again @Jag96 (I'm writing tests next time 😭 )

[Jag96]

Auth PR was merged, after the Auth deploy this will be resolved

[AndrewGable]

@Jag96 / @marcaaron - Can you confirm this fix is live since Auth was deployed and we can close this?

[marcaaron]

Yes, it's working for me now!