-
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
Undo conversion of errorAttemptsCount to object #4939
Conversation
Off hold! |
🚀 Deployed to staging by @MonilBhavsar in version: 1.0.92-2 🚀
|
@NikkiWines Hello! I'm getting the error message in a modal instead of a Growl, I was comparing with the video you attached in the main comment but not sure if this has to do with the PR or not |
cc @aldo-expensify this is probably expected and related to the changes we are making? |
I believe it's fine @isagoico since I know we were moving away from growls for bank account set up, but let's wait for @aldo-expensify to confirm just in case! |
Pretty confident that:
So in lieu of input from @aldo-expensify I'm going to check this off. |
This is expected behaviour, we changed that growl for the modal here |
🚀 Deployed to production by @roryabraham in version: 1.0.93-1 🚀
|
cc: @marcaaron
Details
Follow up PR to https://github.com/Expensify/Web-Secure/pull/2022 to clean out some old, no longer needed logic.
Fixed Issues
$ #4889 (comment)
Tests
isPlaidTestAccountNumbers()
andisPlaidTestRequestor()
andin
lib/ACHData.phpto return
false` so that we run the some necessary checksQA
/bank-account
Connect Manually
and enter the credentials for aPENDING
bank account from this SO postRequestor
step confirm that you get thePlease verify your name and date of birth. If the information is correct, click "Save & Continue
error onceTested On
Screenshots
Web
Screen.Recording.2021-08-30.at.5.30.13.PM.mov
Mobile Web
Desktop
iOS
Android