[$250] LHN - LHN displays employee name instead of workspace name and vice versa

[lanitochka17]

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: 9.0.41-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause - Internal Team

Action Performed:

Option 1:

1. Navigate to https://staging.new.expensify.com/
2. As User A, create a new workspace
3. Go to the Members section and add User B as an employee
4. Log in as User B and go to the Inbox, then observe the workspace created by User A
5. In the LHN , click on the workspace chat

Option 2 (If Option 1 doesn't reproduce the issue):
Follow steps 1-3 above
4. As User A, go to the Inbox and observe the employee workspace chat
5. In the LHN, click on the employee's workspace chat

Expected Result:

• Option 1: The LHN should display the workspace name in the Inbox, not the employee's name
• Option 2: The LHN should display the employee name in the workspace chat, not the workspace name

Actual Result:

• Option 1: In step 4, the LHN initially displays User B's(employee's) name instead of the workspace name. After clicking on it, the name changes to the correct workspace name
• Option 2: In step 4, the LHN initially displays the workspace name instead of User B's name. After clicking on it, the name changes to the employee name (User B)

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

Add any screenshot/video evidence

Bug6619386_1727621597224.2.mp4

Bug6619386_1727621597219.1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021840938166093058581
• Upwork Job ID: 1840938166093058581
• Last Price Increase: 2024-10-01
• Automatic offers:
• ZhenjaHorbach | Reviewer | 104228120

Issue Owner

Current Issue Owner: @bondydaa

[melvin-bot]

Triggered auto assignment to @jliexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[lanitochka17]

@jliexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[melvin-bot]

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

[melvin-bot]

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

[MelvinBot]

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

[jliexpensify]

Hmm I can't seem to reliqably reproduce this. @ZhenjaHorbach - any luck? If you can, we can look at fixing this.

[ZhenjaHorbach]

Hmm I can't seem to reliqably reproduce this. @ZhenjaHorbach - any luck? If you can, we can look at fixing this.

I'll try to reproduce it today !

[jliexpensify]

Just a heads up @ZhenjaHorbach - I am OOO from the 3rd to 14th. I won't reassign this as I don't think any payments will be needed during this time.

[ZhenjaHorbach]

Hmmmm
On the second try I was able to reproduce it ! (Used new accounts)

And looks like FE issue since when we get MissingOnyxMessages we have all information
but I may be wrong
I'll check in more detail today or tomorrow

[jliexpensify]

Nice, ty! Sorry - to clarify, you think this will be a back-end issue?

[ZhenjaHorbach]

Nice, ty! Sorry - to clarify, you think this will be a back-end issue?

I haven't checked this issue in detail yet
But to me it looks like FE issue !

[narefyev91]

Hi, I'm Nicolay from Callstack - expert contributor group - and I would like to work on this issue.

[melvin-bot]

📣 @ZhenjaHorbach 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

[jliexpensify]

All yours @narefyev91 - thanks!

[narefyev91]

Updated from my side - still investigating - but potentially can be BE issue. Will run some more tests and provide tomorrow my findings

[narefyev91]

My discovers so far:
Option 1:
When some invite user A to the workspace - user A get latest information from MissingOnyxMessages API call.
If we observe response which we get from MissingOnyxMessages API call - and compare it with API call when user directly click on chat - we will see that we have just one field difference:

In the code - isOwnPolicyExpenseChat is exact prop - based on which we rendering correct/incorrect header:

Seems like the root case here - API returns not correct value for isOwnPolicyExpenseChat it should be true instead of false.

Option 2:
When user A as an owner of the workspace invite member to his workspace - we fire API call - AddMembersToWorkspace - which returns isOwnPolicyExpenseChat = true:

When user A clicks directly on chat - we get from OpenReport - isOwnPolicyExpenseChat = false
Again this is one main difference between API calls:

And keep in mind that isOwnPolicyExpenseChat prop is directly influence on chat header - problem again coming from not correct isOwnPolicyExpenseChat value - it should be false instead of true.

Why the issue is happened for Option 2 just for new workspaces - answer is simple:
When you create a workspace, immediately invite member to your workspace and go back to main screen - you will automatically landed on Welcome to Admins screen (because you create a new workspace)

When user work with existed workspace, invite user and move back to main screen - user will automatically landed on new created chat (with invited member), OpenReport API will be called - and correct isOwnPolicyExpenseChat value will be set.

@ZhenjaHorbach let me know wdyt here.

[ZhenjaHorbach]

My discovers so far: Option 1: When some invite user A to the workspace - user A get latest information from MissingOnyxMessages API call. If we observe response which we get from MissingOnyxMessages API call - and compare it with API call when user directly click on chat - we will see that we have just one field difference: In the code - isOwnPolicyExpenseChat is exact prop - based on which we rendering correct/incorrect header: Seems like the root case here - API returns not correct value for isOwnPolicyExpenseChat it should be true instead of false.

Option 2: When user A as an owner of the workspace invite member to his workspace - we fire API call - AddMembersToWorkspace - which returns isOwnPolicyExpenseChat = true: When user A clicks directly on chat - we get from OpenReport - isOwnPolicyExpenseChat = false Again this is one main difference between API calls: And keep in mind that isOwnPolicyExpenseChat prop is directly influence on chat header - problem again coming from not correct isOwnPolicyExpenseChat value - it should be false instead of true.

Why the issue is happened for Option 2 just for new workspaces - answer is simple: When you create a workspace, immediately invite member to your workspace and go back to main screen - you will automatically landed on Welcome to Admins screen (because you create a new workspace) When user work with existed workspace, invite user and move back to main screen - user will automatically landed on new created chat (with invited member), OpenReport API will be called - and correct isOwnPolicyExpenseChat value will be set.

@ZhenjaHorbach let me know wdyt here.

Oh
Thanks for the detailed investigation
I'll check it out over the weekend !

[jliexpensify]

Posted in the #convert room for a new Engineer: https://expensify.slack.com/archives/C07HPDRELLD/p1731896542353939

[jliexpensify]

Bumped thread again

[ZhenjaHorbach]

@jliexpensify
I can add a random engineer using 🎀🎀

[jliexpensify]

@ZhenjaHorbach could you do that? It's probably easier than trying to source an Engineer through the Slack channel 😅

[ZhenjaHorbach]

🎀👀🎀

[melvin-bot]

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

[melvin-bot]

@narefyev91, @jliexpensify, @jasperhuangg, @ZhenjaHorbach Whoops! This issue is 2 days overdue. Let's get this updated quick!

[jliexpensify]

Jasper is OOO

[melvin-bot]

@narefyev91, @jliexpensify, @jasperhuangg, @ZhenjaHorbach Huh... This is 4 days overdue. Who can take care of this?

[ZhenjaHorbach]

Not overdue
But if needed I can add a new random engineer

[jliexpensify]

@jasperhuangg - just checking, you're back from medical leave right?

[jasperhuangg]

Sorry for the delay here, been catching up with other issues after coming back from OOO. I was not on medical leave btw, just on holiday 😆

[jasperhuangg]

Have other issues I still need to work on that take priority over this

[jliexpensify]

Ah sorry about that! Shall we drop this to a weekly then, or you want to keep at daily for focus?

[melvin-bot]

@narefyev91, @jliexpensify, @jasperhuangg, @ZhenjaHorbach Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!