LHN- "You don't have access to this chat" message is not displayed after change the URL to http://staging.new.expensify.com/r/hello

[kbecciv]

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

---

Issue found when executing PR #10452

Action Performed:

1. Go to staging.new.expensify.com
2. Log in with your account
3. Navigate to a report url with an invalidID like http://staging.new.expensify.com/r/hello, check you see the You don't have access to this chat message

Expected Result:

User can see the "You don't have access to this chat" message after navigate to a report url with an invalidID like http://staging.new.expensify.com/r/hello

Actual Result:

"You don't have access to this chat" message is not displayed after change the URL to http://staging.new.expensify.com/r/hello

Workaround:

Unknown

Platform:

Where is this issue occurring?

• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.2.11.1

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): any

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

Notes/Photos/Videos: Any additional supporting documentation

Screen_Recording_20221003-104143_Chrome.1.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

[yuwenmemon]

Ah, this seems similar to the drawer closing thing that @marcaaron and @luacmartins were talking about here

[yuwenmemon]

And then later identified by @chiragsalian here as well

[yuwenmemon]

@chiragsalian based on the above am I correct in thinking that this issue is to be handled as a part of the Comment Linking solution?

[chiragsalian]

Ah yup the issue here is that closing the drawer on a reportID or reportActionID doesn't work currently and yes this will be tackled in comment linking. I'd say to just close this issue since its a known problem which will be tackled in comment linking.

[kbecciv]

QA team is having problem with it on build 1.2.30.0

Record_2022-11-23-10-21-39.mp4

[chiragsalian]

well r/1234 does not sound like a valid reportID so routing to the LHN actually looks good to me 😅

And if you do enter a valid reportID, the drawer will still close so yes it's a valid problem but it's something that will be tackled in the comment linking issue.