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

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

Closed
kbecciv opened this issue Oct 14, 2022 · 14 comments
Closed
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Improvement Item broken or needs improvement. Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review

Comments

@kbecciv
Copy link

kbecciv commented Oct 14, 2022

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

Bug5777457_Bandicam_2022-10-14_22-28-54-779.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@melvin-bot
Copy link

melvin-bot bot commented Oct 14, 2022

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

@melvin-bot
Copy link

melvin-bot bot commented Oct 18, 2022

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

@MonilBhavsar MonilBhavsar added the Improvement Item broken or needs improvement. label Oct 18, 2022
@MonilBhavsar
Copy link
Contributor

Draft PR is up, working on tests now

@melvin-bot
Copy link

melvin-bot bot commented Oct 31, 2022

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

1 similar comment
@melvin-bot
Copy link

melvin-bot bot commented Oct 31, 2022

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

@thesahindia
Copy link
Member

Hey @MonilBhavsar, can we assign a CM for C+ compensation (for reviewing #11959).

@melvin-bot
Copy link

melvin-bot bot commented Nov 2, 2022

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

@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@MonilBhavsar MonilBhavsar added the Internal Requires API changes or must be handled by Expensify staff label Nov 3, 2022
@MonilBhavsar
Copy link
Contributor

Hey @MonilBhavsar, can we assign a CM for C+ compensation (for reviewing #11959).

Yes! 👍

@MonilBhavsar MonilBhavsar added Bug Something is broken. Auto assigns a BugZero manager. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Nov 3, 2022
@melvin-bot
Copy link

melvin-bot bot commented Nov 3, 2022

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

@MonilBhavsar
Copy link
Contributor

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

@adelekennedy
Copy link

@thesahindia hired you here

@thesahindia
Copy link
Member

Thanks, accepted the offer!

@adelekennedy
Copy link

paid!

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. Daily KSv2 Engineering Improvement Item broken or needs improvement. Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests

6 participants