Skip to content
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

Merged
merged 2 commits into from
Sep 3, 2021

Conversation

NikkiWines
Copy link
Contributor

@NikkiWines NikkiWines commented Aug 31, 2021

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

  1. Set isPlaidTestAccountNumbers() and isPlaidTestRequestor() and inlib/ACHData.phpto returnfalse` so that we run the some necessary checks
  2. Run through the QA steps

QA

  1. Log into an account on new.expensify.com without a bank account set up
  2. Navigate to /bank-account
  3. Click on Connect Manually and enter the credentials for a PENDING bank account from this SO post
    • image
  4. Once you get to the Requestor step confirm that you get the Please verify your name and date of birth. If the information is correct, click "Save & Continue error once
  5. Resubmit your information and confirm you're routed to the next view and not looped back to the requestor step again

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Screen.Recording.2021-08-30.at.5.30.13.PM.mov

Mobile Web

Desktop

iOS

Android

@NikkiWines NikkiWines requested a review from a team as a code owner August 31, 2021 00:48
@NikkiWines NikkiWines self-assigned this Aug 31, 2021
@MelvinBot MelvinBot requested review from MonilBhavsar and removed request for a team August 31, 2021 00:48
@NikkiWines NikkiWines changed the title [HOLD] Undo conversion of errorAttemptsCount to object Undo conversion of errorAttemptsCount to object Sep 2, 2021
@NikkiWines
Copy link
Contributor Author

Off hold!

@MonilBhavsar MonilBhavsar merged commit 7eb2847 into main Sep 3, 2021
@MonilBhavsar MonilBhavsar deleted the nikki-remove-errorAttemptsCount-overwrite branch September 3, 2021 08:54
@OSBotify
Copy link
Contributor

OSBotify commented Sep 3, 2021

🚀 Deployed to staging by @MonilBhavsar in version: 1.0.92-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@isagoico
Copy link

isagoico commented Sep 3, 2021

@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
image

@marcaaron
Copy link
Contributor

cc @aldo-expensify this is probably expected and related to the changes we are making?

@NikkiWines
Copy link
Contributor Author

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!

@roryabraham
Copy link
Contributor

Pretty confident that:

  1. This is expected behavior
  2. If unexpected, it's not caused by this PR.

So in lieu of input from @aldo-expensify I'm going to check this off.

@aldo-expensify
Copy link
Contributor

This is expected behaviour, we changed that growl for the modal here

@OSBotify
Copy link
Contributor

OSBotify commented Sep 4, 2021

🚀 Deployed to production by @roryabraham in version: 1.0.93-1 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants