-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Bank account - EUR bank account setup flow can be proceeded with USD bank account setup info #60518
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 @CortneyOfstad ( |
Triggered auto assignment to @robertjchen ( |
💬 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 |
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. |
thanks @MrMuzyk ! |
Because there were other issues with the flow and they're all relatively small we've decided to fix all of them in one big follow-up issue - #60633 |
thanks, we can consolidate there 👍 |
On friday @madmax330 did let us know on slack that the issue is fixed and should be deployed today. I'll retest it today/tomorrow |
Sounds good @MrMuzyk! If you need any help with testing, please let us know! |
Flow gets reset properly when user specifically requests it but when it should happen in the background (on currency change) it still doesn't work. I've let Max know on slack and waiting for further info. |
Thanks @MrMuzyk! |
@robertjchen, @burczu, @CortneyOfstad, @MrMuzyk Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
I'm still waiting on signal from @madmax330 that it's fixed |
Not overdue |
@robertjchen, @burczu, @CortneyOfstad, @MrMuzyk Whoops! This issue is 2 days overdue. Let's get this updated quick! |
I think this is fixed now? cc: @madmax330 🙏 |
@madmax330 can you confirm? 🤞 |
Dropped Max a DM 👍 |
No this is still not fixed, but I'm working on the PR. Should be done early next week |
Thanks Maxence 🙇 |
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: Mac 15.3 / Chrome
App Component: Workspace Settings
Action Performed:
Precondition:
Expected Result:
In Step 11, app should reset the USD bank account setup flow because USD and EUR bank account setup flows contain different required information.
Actual Result:
In Step 11, app allows user to continue EUR bank acconunt setup using USD bank account setup information.
In Step 12, when returning to previous page during EUR bank account setup, many fields are blank.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6806807_1745017737200.20250419_065654.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: