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

Amount validation is requested on Connect with Plaid after trying to add BA manually #37865

Closed
2 of 6 tasks
m-natarajan opened this issue Mar 7, 2024 · 7 comments
Closed
2 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Hourly KSv2

Comments

@m-natarajan
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 1.4.48-0
Reproducible in staging?: y
Reproducible in production?: couldn't check
If this was caught during regression testing, add the test name, ID and link from TestRail: n/a
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:

Action Performed:

Pre-requisite: user must be logged in and have created a workspace.

  1. Go to Workspace > Bank account.
  2. Select Connect manually.
  3. Initiate the add bank account flow with the testing credentials until you reach the "Validate your bank account" page.
  4. Click on the back arrow.
  5. Click on "Start over".
  6. Select Connect online with Plaid.
  7. Select Regions bank and use the testing credentials.
  8. Confirm the account.
  9. Verify you land on "Validate your bank account" page.

Expected Result:

The user should be able to continue with the add BA flow (enter Personal info, Company info, Beneficial owners).

Actual Result:

The user is taken to "Validate your bank account" page after confirming the account.

Workaround:

unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6404818_1709777237768.bandicam_2024-03-06_21-51-37-512.mp4

View all open jobs on GitHub

@m-natarajan m-natarajan added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 7, 2024
Copy link

melvin-bot bot commented Mar 7, 2024

Triggered auto assignment to @MitchExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Mar 7, 2024
Copy link
Contributor

github-actions bot commented Mar 7, 2024

👋 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:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Mar 7, 2024

Triggered auto assignment to @techievivek (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@m-natarajan
Copy link
Author

@techievivek FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

@MitchExpensify
Copy link
Contributor

Related? #32356

@nkuoch nkuoch removed the DeployBlockerCash This issue or pull request should block deployment label Mar 7, 2024
@nkuoch nkuoch self-assigned this Mar 7, 2024
@nkuoch
Copy link
Contributor

nkuoch commented Mar 7, 2024

It's a feature, not a bug.

Disconnecting from a policy when in PENDING or OPEN state doesn't remove it from the account anymore. It just detaches it from the policy. So, when re-entering the same bank account, you don't need to redo it all. It just attaches the existing one.

If you actually delete it from your account from your personal settings, you'll have to redo it all, as before.

@nkuoch nkuoch closed this as completed Mar 7, 2024
@MitchExpensify
Copy link
Contributor

@m-natarajan seeing as this is a feature not a bug, do you agree the TR steps need to be updated to avoid this being reported again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Hourly KSv2
Projects
None yet
Development

No branches or pull requests

4 participants