[$2000] Web/mweb - Chat - LHN does not display after signing out/signing in

[kbecciv]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Issue found when executing PR #11006

Action Performed:

1. Go to staging.new.expensify.com
2. Login with user applausetester+0901abb@applause.expensifail.com, using the password Feya86Katya

Expected Result:

Chat list is loading without refreshing the page

Actual Result:

Chat list displays after refreshing

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• Mobile Web
• Desktop

Version Number: 1.2.0.4

Reproducible in staging?: Yes

Reproducible in production?: No

Email or phone of affected tester (no customers): applausetester+0901abb@applause.expensifail.com/Feya86Katya

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

Notes/Photos/Videos: Any additional supporting documentation

11006.mWeb.2.mp4

Recording.1217.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

[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.

[melvin-bot]

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

[Julesssss]

I can't reproduce this on v1.2.0-4, with my own account, or with the tester credentials provided. Tested on web (Chrome) and mWeb (Android).

[kbecciv]

Adding Desktop as well for environment list

[Julesssss]

Asked here for someone else to confirm, as I wasn't able to reproduce

[luacmartins]

I was able to reproduce but not consistently

[mountiny]

Updating the title to be more accurate.

[johnmlee101]

This happens while OpenApp is processing, and the result of OpenApp looks fine, so its the render that follows thats failing?

[francoisl]

I can consistently reproduce on web with the account applausetester+0901abb@applause.expensifail.com. Should we just revert #10784 for now?

[luacmartins]

@francoisl can we confirm that reverting #10784 fixes the issue?

[francoisl]

No because I don't think anyone was able to reproduce the issue in dev in the first place, but the idea is:

• this blocker was introduced by Omit default Avatar param to prevent crash on app start #11006
• Omit default Avatar param to prevent crash on app start #11006 was an attempt to fix an issue introduced by Optimize report ordering in the LHN #10784

[johnmlee101]

We'd have to revert the CP as well if we go this route, right?

[francoisl]

Yes we'd either revert the CP first, or there would be a conflict if we directly revert #10784. We can do both reverts in one PR though, it's not really a problem.

[johnmlee101]

Okay I'd say to prevent blocking another deploy let's do that. Order of operations is

1. Create the revert for Omit default Avatar param to prevent crash on app start #11006
2. Add the revert for Optimize report ordering in the LHN #10784 into the mix
3. CP to staging label
4. Test

Correct?

[francoisl]

Yes and we can make steps 1 and 2 in one PR - give me a second to double check.

[francoisl]

We can reproduce this in production too - it might be related to a recent Web-E change though, let's keep this open as a high-priority issue.

[luacmartins]

lol why does Melvin keep assigning internal engineers? Am I the CME for this issue?

[mallenexpensify]

Yeah, @luacmartins , you're the CME, John and Francois aren't on the CME team.

[Christinadobrzyn]

Created job in Upwork:

Internal posting: https://www.upwork.com/ab/applicants/1577530367682449408/job-details
External posting: https://www.upwork.com/jobs/~01f8b292800ec04b69

[Christinadobrzyn]

Waiting for proposals

[melvin-bot]

@francoisl, @johnmlee101, @eVoloshchak, @luacmartins, @Christinadobrzyn Whoops! This issue is 2 days overdue. Let's get this updated quick!

[luacmartins]

Gonna move this to a weekly while we wait for proposals.

[Christinadobrzyn]

Doubling price to $500

[Christinadobrzyn]

doubled price - still waiting on proposals

[luacmartins]

Still looking for proposals

[Christinadobrzyn]

Price doubled $2000 - still waiting on proposals

[luacmartins]

I'm unable to reproduce this. @kbecciv could you please ask the team to try to reproduce on the latest build?

[kbecciv]

Issue is not reproduced in build v1.2.23-9

Recording.1611.mp4

[luacmartins]

Thanks @kbecciv! Ok, I'm gonna close this since we can't reproduce the issue. My guess is that the performance improvements we made to the LHN fixed this somehow. Please reopen if this happens again.