Open lanitochka17 opened 3 weeks ago
Triggered auto assignment to @sonialiap (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
@sonialiap Whoops! This issue is 2 days overdue. Let's get this updated quick!
Job added to Upwork: https://www.upwork.com/jobs/~01dbdf127f2f018440
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav (External
)
From more features page tapping back user directed to distance rate instead settings
In https://github.com/Expensify/App/blob/main/src/pages/workspace/distanceRates/PolicyDistanceRatesSettingsPage.tsx#L143-L144, the app uses navigate
so the Distance Rate page will still remain in the navigation stack, so when go back it will go back to it.
Navigation.goBack(ROUTES.WORKSPACE_MORE_FEATURES.getRoute(policyID));
Doing it this way will make sure the Distance Rate page was popped from the navigation stack and when going back from More Features page, it will correctly go back to the Settings page
Optional: 2nd param shouldEnforceFallback
could be put as true
Use Navigation.resetToHome();
to reset the navigation state correctly, after that navigate to the more features page
So this will be updated to
Navigation.resetToHome();
Navigation.navigate(ROUTES.WORKSPACE_MORE_FEATURES.getRoute(policyID));
Another idea is to manually pop the navigation route until we meet the Settings page (can be as simple as going back twice first), then navigate to more features page
I updated proposal to add an alternative solution.
Will review this today. Will need to check the current navigation screens.
I think @dominictb's solution would work using goBack
.
π π π C+ reviewed
Triggered auto assignment to @flodnv, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
π£ @dominictb π An offer has been automatically sent to your Upwork account for the Contributor role π Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review π§βπ» Keep in mind: Code of Conduct | Contributing π
If you havenβt already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1. 4.83 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: N/A Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
From more features page tapping back user must be directed to settings page
Actual Result:
From more features page tapping back user directed to distance rate instead settings
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/78819774/992778dd-b05c-43c4-b23e-a91bed2b75d9
View all open jobs on GitHub
Upwork Automation - Do Not Edit