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

Wallet - Wallet cannot be enabled after Onfido identify #43709

Closed
1 of 6 tasks
lanitochka17 opened this issue Jun 13, 2024 · 14 comments
Closed
1 of 6 tasks

Wallet - Wallet cannot be enabled after Onfido identify #43709

lanitochka17 opened this issue Jun 13, 2024 · 14 comments
Assignees

Comments

@lanitochka17
Copy link

lanitochka17 commented Jun 13, 2024

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.83-0
Reproducible in staging?: Y
Reproducible in production?: Unable to check
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Issue found when executing PR #42589

Action Performed:

  1. Login to ND
  2. GO to Account Settings > Wallet > enable
  3. Proceed the flow with name Alberta Charleson
  4. Complete Onfido identification

Expected Result:

Bank account added. No message about impossibility of enabling wallet

Actual Result:

Bank account added. Message 'Unfortunately your wallet cannot be enabled at this time. Please chat with Concierge for further assistance.' is displayed under wallet balance

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

Bug6512180_1718296305520.wallet_1.mp4
Bug6512180_1718296305542.wallet_2.mp4

Bug6512180_171

View all open jobs on GitHub

@lanitochka17 lanitochka17 added DeployBlockerCash This issue or pull request should block deployment DeployBlocker Indicates it should block deploying the API labels Jun 13, 2024
Copy link

melvin-bot bot commented Jun 13, 2024

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

Copy link
Contributor

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

@lanitochka17
Copy link
Author

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

@puneetlath
Copy link
Contributor

Seems like likely just a one-off because we fail if Onfido takes more than 6 seconds to respond. Asked for a re-test.

@luacmartins luacmartins removed the DeployBlockerCash This issue or pull request should block deployment label Jun 13, 2024
@kavimuru
Copy link

kavimuru commented Jun 13, 2024

@puneetlath tester able to reproduce with new account applausetester+bp1306w@applause.expensifail.com

WLT_2.mp4
WLT_1.mp4

@NikkiWines
Copy link
Contributor

Demoting per this discussion

@NikkiWines NikkiWines added Daily KSv2 and removed DeployBlocker Indicates it should block deploying the API Hourly KSv2 labels Jun 13, 2024
@melvin-bot melvin-bot bot added the Overdue label Jun 17, 2024
Copy link

melvin-bot bot commented Jun 17, 2024

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

@puneetlath
Copy link
Contributor

@lanitochka17 are you still seeing this every time you attempt to set up a bank account?

@melvin-bot melvin-bot bot removed the Overdue label Jun 18, 2024
@puneetlath puneetlath added Weekly KSv2 and removed Daily KSv2 labels Jun 18, 2024
@lanitochka17
Copy link
Author

Issue is reproducible
Reproducible all time

bandicam.2024-06-19.17-50-07-608.1.mp4

@puneetlath
Copy link
Contributor

@MariaHCD @nkuoch would y'all happen to have any idea why this might happen when @lanitochka17 is testing with the test bank account? When I tried with my own real bank account it worked fine.

@MariaHCD
Copy link
Contributor

We currently do not support creating test wallets (i.e. wallets set up with a test bank account or with the Alberta Charleson test details).

Test wallets will be supported as part of https://github.com/Expensify/Expensify/issues/364990 but it's currently not a top priority.

@puneetlath
Copy link
Contributor

puneetlath commented Jun 25, 2024

Ah ok. So if I understand correctly @MariaHCD. Then what @lanitochka17 is experiencing is expected. Is that right?

@MariaHCD
Copy link
Contributor

Yes, this is expected for now 👍🏼

@puneetlath
Copy link
Contributor

Ok I'm going to close then. @lanitochka17 can you update whatever documentation is needed on your side to account for this?

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

No branches or pull requests

6 participants