-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deeplinks - "doesn't have access" page instead of LHN #17390
Comments
Triggered auto assignment to @MitchExpensify ( |
Bug0 Triage Checklist (Main S/O)
|
Linking this Hold KI #12676 |
This is not actually expected behavior as far as I know: "User should be redirected to the to the LHN in each link" When you try access a report that you don't have access to we expect to be shown the "Hmm.. it's not here" page. CC @jasperhuangg and @thienlnam as you are both close to these flows 😄 |
Yeah this seems like it is working correctly? Like unless the user actually has access to those reports they should be seeing that "Hmm.. it's not here" unless it's a public room |
Agreed! Closing |
Going to re-open so we can address the TR case where this happens - @isagoico Could we please get this updated to Changes: |
Oh whoops, yes good call @thienlnam |
Lil bump on this please 😄 |
@MitchExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
From this slack message, updating TR steps are getting distributed - @mvtglobally could you please help us take care of this update? |
Will check it out today |
@thienlnam Let me know if there is anything else required to be added here |
Looks great, thanks! |
@MitchExpensify Be sure to fill out the Contact List! |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
https://staging.new.expensify.com/r/6821704960193694/details
https://staging.new.expensify.com/r/6821704960193694/participants
https://staging.new.expensify.com/r/6821704960193694/settings
Expected Result:
User should be redirected to the to the LHN in each link
Actual Result:
User is redirected to "doesn't have access" page
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.2.99.3
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
https://expensify.testrail.io/index.php?/tests/view/3363141
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Bug6016476_video_06__1_.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: