[$500] Room - Hmm... it's not here page appears when leaving the room on another device

[lanitochka17]

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 #24111

Action Performed:

1. Go to https://staging.new.expensify.com/
2. Login with any Expensifail account in 2 different device
3. Create the room with WS that have some members
4. Send a simple text message, a picture and request to split bill
5. Exit the room on another device

Expected Result:

You get navigated to concierge

Actual Result:

Hmm... it's not here page appears

Workaround:

Unknown

Platforms:

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

• Android / native
• Android / Chrome
• iOS / native
• iOS / Safari
• Windows / Chrome
• MacOS / Desktop

Version Number: 1.3.58-0

Reproducible in staging?: Yes

Reproducible in production?: Yes

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

Notes/Photos/Videos: Any additional supporting documentation

Recording.6053.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01a94d6a85e6b14ed3
• Upwork Job ID: 1699144974222155776
• Last Price Increase: 2023-09-05

[melvin-bot]

Triggered auto assignment to @johncschuster (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]

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

[melvin-bot]

@johncschuster Still overdue 6 days?! Let's take care of this!

[melvin-bot]

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

[melvin-bot]

Current assignee @johncschuster is eligible for the External assigner, not assigning anyone new.

[melvin-bot]

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

[melvin-bot]

📣 @sobitneupane Please request via NewDot manual requests for the Contributor role ($500)

[melvin-bot]

@johncschuster @sobitneupane this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[melvin-bot]

@johncschuster, @sobitneupane Huh... This is 4 days overdue. Who can take care of this?

[melvin-bot]

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

[melvin-bot]

@johncschuster, @sobitneupane Eep! 4 days overdue now. Issues have feelings too...

[sobitneupane]

@pradeepmdk The PR you mentioned has merged. Is this issue still reproducible in your end?

[pradeepmdk]

@sobitneupane It is working now as expected.

Screen.Recording.2023-09-25.at.9.08.35.PM.mov

@sobitneupane some blink is there before navigating to navigateToConciergeChat Can we fix it here?

[sobitneupane]

some blink is there before navigating to navigateToConciergeChat Can we fix it here?

@pradeepmdk Yes. I think we should fix it. Do you have proposal?

[melvin-bot]

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

[melvin-bot]

@johncschuster, @sobitneupane Still overdue 6 days?! Let's take care of this!

[johncschuster]

bump @pradeepmdk!

[pradeepmdk]

@johncschuster Sorry I missed the notification. let me update you now.

[pradeepmdk]

@johncschuster @sobitneupane this issue was pusher sending delay response. before sending USER_IS_LEAVING_ROOM its send the multipleEvents so that its showing Hmm...page after getting the USER_IS_LEAVING_ROOM events its redirected to ConciergeChat.

but at the code level we are sending the USER_IS_LEAVING_ROOM first even though its sending the multipleEvents first.

App/src/libs/actions/Report.js

Line 1916 in 035ee44

broadcastUserIsLeavingRoom(reportID);

Screen.Recording.2023-10-05.at.9.26.59.PM.mov

so currently i don't have any idea how to boost USER_IS_LEAVING_ROOM pusher event first.

Note: sometime USER_IS_LEAVING_ROOM is triggered first at the time we don't have this issue.

Screen.Recording.2023-10-05.at.9.29.46.PM.mov

[melvin-bot]

@johncschuster, @sobitneupane Huh... This is 4 days overdue. Who can take care of this?

[johncschuster]

@sobitneupane bump!

[sobitneupane]

Thanks for the update @pradeepmdk. Appreciate your help in the issue.

[sobitneupane]

The nature of the issue has changed a bit. The "Hmm... it's not here page" appears only for short duration. We are looking on proposals to see if we can get rid of the Not found page.

[johncschuster]

Thanks for that, @sobitneupane! How would you re-title this issue to better reflect what we're looking for?

Also, do the reproduction steps still match?

(I'll update the OP and post for proposals)

[melvin-bot]

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

[ayazalavi]

The nature of the issue has changed a bit. The "Hmm... it's not here page" appears only for short duration. We are looking on proposals to see if we can get rid of the Not found page.

This bug is not reproducible anymore, page is directly going to concierage page.

[johncschuster]

Thanks, @ayazalavi! It sounds like this behavior may have been resolved by a different issue/PR. I'm going to go ahead and close it.