-
Notifications
You must be signed in to change notification settings - Fork 3k
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
mWeb - Expensify Card - Unable to navigate back to settings after Reporting card as Fraud #57297
Comments
Triggered auto assignment to @jliexpensify ( |
Triggered auto assignment to @grgia ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
ProposalPlease re-state the problem that we are trying to solve in this issue.User is not able to navigate back from Wallet and Card pages What is the root cause of that problem?When clicking back button in Card page, we call
And when clicking back button in Wallet page, we call
This traps us in a loop: navigate to wallet page -> go back to card page -> navigate to wallet page .... What changes do you think we should make in order to solve the problem?We should use
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?None What alternative solutions did you explore? (Optional)Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job. |
I'm unsure if this is an actual deploy blocker but it seems like an issue - @grgia will know best here. |
@jliexpensify does this occur on prod? |
@grgia I believe I can reproduce it on Android Hybrid app (9.1.4.0) without even cancelling a card. Repro steps:
But if I use swipe gestures, I can navigate back - but not by hitting the back button with my finger/thumb. Video: screen-20250224-204503.mp4 |
So pretty much @daledah's comment here:
|
Ugh ignore me...that's staging 🤦 EDIT: I can't repro on prod (Android) |
We have reverted potentially related PR cc @WojtekBoman #57320 this might not be reproducible after this revert |
I'm monitoring this issue, so if you can or cannot reproduce this then please write a comment. If someone needs to work on this then we might help, I have some partial context related to this PR |
Asked for a retest https://expensify.slack.com/archives/C9YU7BX5M/p1740408684514249 |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.1.4-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Yes, reproducible on both
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5640930
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: iPhone 12 Pro / iOS 17.4.1
App Component: Workspace Settings
Action Performed:
3 Navigate to Wallet
Expected Result:
User should navigate back to Settings
Actual Result:
User is not able to navigate back from Wallet and Card pages
Workaround:
Unknown
Platforms:
Screenshots/Videos
https://github.com/user-attachments/assets/f10add12-ecc6-49bb-9018-bbebce160601
https://github.com/user-attachments/assets/4fda9ffe-e4a0-4a6e-b2c8-7bc1378880d5
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: