-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-05-22] [$250] Android - Device back button reopens bank account setup after it is closed via app back button #60509
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
Comments
Triggered auto assignment to @laurenreidexpensify ( |
Triggered auto assignment to @marcaaron ( |
💬 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:
|
Not a blocker. GR is under beta cc @burczu |
Coming from the perspective of the Web Deploy, I removed the
|
Job added to Upwork: https://www.upwork.com/jobs/~021914413054452390121 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @suneox ( |
I am Michał from Callstack - expert contributor group. I’d like to work on this job. Im also one of the authors of this whole flow this is now behind beta so I have all the needed context. |
@mitarachim @laurenreidexpensify Hi, I wanted to ask and confirm expected result here. Wouldn't it make more sense for the back arrow on the overview screen to behave the same way that device's back button does? It seems a little bit weird to me that suddenly back arrow doesn't lead user back to previous screen but rather some different path. Video showcasing what I have in mind below. Screen.Recording.2025-04-25.at.15.00.26.mov |
@Expensify/design can you confirm your understanding of the expected pattern here? Thanks |
I think we typically want the device back button to work the same as the browser back button, which should also work similarly to the app back button. Having a tough time following this thread, but generally I think I agree with this comment:
@Expensify/design for a gut check on that though! |
I think what you have in the video above looks right to me. Generally the Android back button is a back to previous visited screen and a back arrow in the app is up the hierarchy, but in this instance we want them to just sidestep to change the currency and then pop back into the flow to continue connecting their bank account so the video is correct in my view 👍 |
Back button on browser behaves the same way as the device back button on my video I've posted so I will update the android/ios behaviour accordingly. Thank you for confirmation, working on a fix now. |
Thanks @dubielzyk-expensify @dannymcclain 🙇♀ |
PR that fixes it just got merged. This will be ready to be retested soon. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.45-21 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-05-22. 🎊 For reference, here are some details about the assignees on this issue:
|
@suneox @laurenreidexpensify @suneox The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
BugZero Checklist:
Bug classificationSource of bug:
Where bug was reported:
Who reported the bug:
Regression Test Proposal Template
Regression Test ProposalPrecondition:Test:Do we agree 👍 or 👎 |
@suneox bump for template to be filled out! |
I haven't worked on the PR @hungvu193 will take over this one |
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.30-0
Reproducible in staging?: Yes
Reproducible in production?: Unable to check
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: #56931
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Samsung Galaxy Z Fold 4 / Android 14
App Component: Workspace Settings
Action Performed:
Precondition:
Expected Result:
App will return to LHN.
Actual Result:
App reopens bank account setup.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6806765_1745013834278.Screen_Recording_20250419_060027_Expensify.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @laurenreidexpensifyThe text was updated successfully, but these errors were encountered: