-
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
[$500] Dev: Web - Click on back button close the RHP instead of opening Details page #27598
Comments
Job added to Upwork: https://www.upwork.com/jobs/~013c7fb2a8b63fc0bf |
Triggered auto assignment to @trjExpensify ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Dev: Web - Click on back button close the RHP instead of opening Details page with the case we reload the PrivateNoteListPage. What is the root cause of that problem?We don't define a fallback route for it here:
So when we reload the page, it will go back to the previous stack and it's RHN. What changes do you think we should make in order to solve the problem?We should define a fallback route for our onBackButtonPress={() => Navigation.goBack(ROUTES.getReportDetailsRoute(report.reportID))} We can also add the fallback route for other What alternative solutions did you explore? (Optional)N/A |
Dupe of #22041 |
Yup, agree with that. Closing! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
It should navigates to Details page
Actual Result:
Close the RHP
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: Dev 1.3.70.5
Reproducible in staging?: n
Reproducible in production?: n
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
Screen.Recording.2023-09-15.at.8.02.04.PM.mov
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1694788314587129
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: