-
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
Remove password requirement for setting up withdrawal account #5545
Conversation
@jasperhuangg is at 20% this week so requesting another reviewer. The Auth PR has been merged so the code can be reviewed and tested on dev, waiting to remove the hold until after the auth deploy. |
Auth PR has been deployed, taking off hold |
@Jag96 Helloo! Following orders from QA steps and tagging you for QA 🙇 |
@Jag96, Is this possibly a regression? On dev, it seems like we're unable to go beyond step 2. The UI doesn't show any error for me but the network request says "401 error", @luacmartins tried the same on his end and got, |
Adding to this, I do see the UI error |
Okay, nvm me. I did an updateRepos all and makeAll and once i retested it worked fine for me on dev. |
Same here, just had to update all the repos! Sorry for the false alarm! |
🚀 Deployed to production by @chiragsalian in version: 1.1.4-0 🚀
|
Details
Per this thread we want to remove the password requirement for adding a withdrawal account. This PR updates the UI to not include the password field for the
Company Step
, and removes the password param for theBankAccount_SetupWithdrawal
API call.Fixed Issues
$ https://github.com/Expensify/Expensify/issues/179271
Tests
Get Started
to go through the VBA flow, follow https://stackoverflow.com/c/expensify/questions/342/525#525Company Information
step, the password field doesn't appear4. Confirm you can submit the form, and that there is no error regarding the password
QA Steps
Tag @Jag96 and I can run through the QA steps for this on staging
Tested On
Screenshots
Screenshot in the tests section