App icon indicating copy to clipboard operation
App copied to clipboard

Web -Payments - Connect Bank account -The user sees a password field when add VBA

Open kbecciv opened this issue 2 years ago • 5 comments

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Go to URL https://staging.new.expensify.com/
  2. Login with any account
  3. Go to Settings > Payment
  4. Click "Add payment method" > Bank account
  5. Choose Fidelity bank, enter username and password
  6. Choose Plaid Checking 0000 or Plaid Saving 1111 6.Enter your password
  7. Click "Continue"
  8. Disable internet connection
  9. Delete VBA
  10. Go back online
  11. Click Add Payment option > Select Bank Account

Expected Result:

The user see only the the loading spinner.

Actual Result:

The user see a password field.

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.2.15.2

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): any

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/93399543/195943489-87cd9720-da30-4d15-b75b-497933fb0a7c.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

kbecciv avatar Oct 14 '22 21:10 kbecciv

Triggered auto assignment to @MonilBhavsar (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

melvin-bot[bot] avatar Oct 14 '22 21:10 melvin-bot[bot]

@MonilBhavsar Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Oct 18 '22 06:10 melvin-bot[bot]

Draft PR is up, working on tests now

MonilBhavsar avatar Oct 18 '22 11:10 MonilBhavsar

@MonilBhavsar Eep! 4 days overdue now. Issues have feelings too...

melvin-bot[bot] avatar Oct 31 '22 07:10 melvin-bot[bot]

@MonilBhavsar Eep! 4 days overdue now. Issues have feelings too...

melvin-bot[bot] avatar Oct 31 '22 08:10 melvin-bot[bot]

Hey @MonilBhavsar, can we assign a CM for C+ compensation (for reviewing https://github.com/Expensify/App/pull/11959).

thesahindia avatar Nov 01 '22 11:11 thesahindia

@MonilBhavsar 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

melvin-bot[bot] avatar Nov 02 '22 08:11 melvin-bot[bot]

Issue not reproducible during KI retests. (First week)

mvtglobally avatar Nov 03 '22 03:11 mvtglobally

Hey @MonilBhavsar, can we assign a CM for C+ compensation (for reviewing https://github.com/Expensify/App/pull/11959).

Yes! 👍

MonilBhavsar avatar Nov 03 '22 05:11 MonilBhavsar

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

melvin-bot[bot] avatar Nov 03 '22 08:11 melvin-bot[bot]

@adelekennedy the issue has already been fixed. We need to compensate C+ - @thesahindia for reviewing the PR

MonilBhavsar avatar Nov 03 '22 08:11 MonilBhavsar

@thesahindia hired you here

adelekennedy avatar Nov 08 '22 05:11 adelekennedy

Thanks, accepted the offer!

thesahindia avatar Nov 08 '22 06:11 thesahindia

paid!

adelekennedy avatar Nov 09 '22 00:11 adelekennedy