[$500] Public room - When the user leaves the room, another chat is displayed

[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: 1.4.43-7
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): [email protected]
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

1. Go to https://staging.new.expensify.com/r/4098419659819887
2. Verify the room chat is displayed
3. Tap on sign in
4. Login with an existing account
5. Go to LHN
6. Open any other chat, like a DM
7. Go to the room chat
8. Tap on the three dots menu and select "Leave"
9. Verify Concierge chat is displayed
10. Tap on the back arrow button
11. Verify the last chat you opened is displayed

Expected Result:

The last chat you opened should not be displayed

Actual Result:

The last chat you opened is displayed after tapping on the back arrow button

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

Bug6387438_1708548771330.Uqvd1974_1_.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01e2bc1f1429ae4674
• Upwork Job ID: 1767609426339414016
• Last Price Increase: 2024-04-02

[melvin-bot]

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

[lanitochka17]

We think that this bug might be related to #vip-vsp
CC @quinthar

[lanitochka17]

@bz 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]

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

[laurenreidexpensify]

I am OOO til from 23 Feb - 4 March and this got assigned after I logged off thanks

[dukenv0307]

Proposal

Please re-state the problem that we are trying to solve in this issue.

• Public room - When the user leaves the room, another chat is displayed

What is the root cause of that problem?

• When leaving room, we have a logic to navigate user to the previous accessed report in here
• But, we also have a logic that will navigate user to concierge report in here. In this one, Report.navigateToConciergeChat() is called.
• The two above logics lead to the bug

What changes do you think we should make in order to solve the problem?

• Solution 1 (Highly recommend): We also can remove the logic that navigate user to previous accessed report in here and let the logic handle it instead to make sure there is one logic that handles the navigation after leaving room and it works properly when user leaves room in other devices

• Solution 2: Or, we can create a global variable, named isLeavingRoom. In here, set it to true. Then in here, updating it to:

            if(!isLeavingRoom) { Report.navigateToConciergeChat(); }

What alternative solutions did you explore? (Optional)

• Because we already had a logic to navigate user to previous accessed report once we leave room, so we just need to make sure the Report.navigateToConciergeChat() in here is not called.

• We can do it by updating this to:

            if(prevReport.chatType !== CONST.REPORT.CHAT_TYPE.POLICY_ROOM){
                Report.navigateToConciergeChat();
            }

[abekkala]

@lanitochka17 your video doesn't seem to match the steps you provided.
Can you please provide a new video using the steps you mention?

Reopen if/when necessary

[dukenv0307]

@abekkala I see that the video matches the steps that @lanitochka17 provided.

@lanitochka17 Can you double-check comment

[lanitochka17]

@abekkala Hello
Full video step by step:

0-02-01-996f28f4104773f83412513e7f48a9be13c42adfd1f93ddc4ce1622855c4e981_c89b34c56dc9e9e.mp4

[melvin-bot]

@abekkala this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[abekkala]

@lanitochka17 Can you check if the fix for this one should hav resolved this issue?

[lanitochka17]

Issue is still reproducible on the latest build 1.4.50-2

Cmzc9073.1.mp4

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

[alitoshmatov]

I am having a hard time reproducing it, I kinda did it at some point. But mainly after step 6 chat room provided in step 1 is not present in my LHN

[abekkala]

@lanitochka17 can you reproduce?

[melvin-bot]

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

[abekkala]

@CortneyOfstad I'm ooo until April 08 - I'll be taking back any issues still Open once I return

---

CURRENT STATUS:

We are still attempting to reproduce this and then receive proposals. It's also been assigned to the #wave-collect project. Curator will decide whether to keep open or close.

[CortneyOfstad]

Not overdue – just waiting for someone to pick this up 👍

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[CortneyOfstad]

@lanitochka17 are you able to recreate this still on your side?

[lanitochka17]

@CortneyOfstad Issue is still reproducible

Cbjb9612.1.mp4

[CortneyOfstad]

@mvtglobally since you are having trouble recreating, I wanted to check in and see if you were experiencing any issues or confusion in regards to the steps above? If you think there is any room for improvement in the steps, please share them here so we can get them updated. Thanks!

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[mvtglobally]

Issue not reproducible during KI retests. (Second week)

[CortneyOfstad]

@lanitochka17 I am unable to recreate this on my side and @mvtglobally is unable to recreate this. Going to close — feel free to reopen if you can show consistent reproduction 👍