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

Multiple UPE - payment method does not support recurring payments error in my account #4396

Closed
harriswong opened this issue Jul 4, 2022 · 2 comments · Fixed by #4427
Closed
Assignees
Labels
category: core WC Payments core related issues, where it’s obvious. component: upe type: bug The issue is a confirmed bug.

Comments

@harriswong
Copy link
Contributor

harriswong commented Jul 4, 2022

Description

This is one of the TODOs from #3968 (comment).

When we use the poc/upe-instances-multiplied branch, this error appears on the my account page:

This payment method does not support recurring payments.

This is due to all payment gateways appearing on the page when only the card element should appear. We'll need to filter out gateways that don't support tokenization when register gateways. We can check if we're on the Add Payment Page and if the payment method is reusable.

This ticket is to investigate and fix the error.

Acceptance Criteria

  • This payment method does not support recurring payments. should not show up.
  • A new payment method can be successfully added.
  • changes made should be merged into poc/upe-instances-multiplied, not develop.
@harriswong harriswong changed the title Multiple UPE - payment method does not support recurring payments error Multiple UPE - payment method does not support recurring payments error in my account Jul 4, 2022
@deepakpathania deepakpathania added component: upe type: bug The issue is a confirmed bug. category: core WC Payments core related issues, where it’s obvious. labels Jul 5, 2022
@mdmoore mdmoore self-assigned this Jul 5, 2022
@waclawjacek
Copy link
Contributor

Will PR #4427 resolve all of the problems described here? If so, can we link it to this issue?

@mdmoore mdmoore linked a pull request Aug 2, 2022 that will close this issue
6 tasks
@mdmoore
Copy link
Member

mdmoore commented Aug 2, 2022

Will PR #4427 resolve all of the problems described here? If so, can we link it to this issue?

It will. PR linked ✅

@mdmoore mdmoore closed this as completed Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: core WC Payments core related issues, where it’s obvious. component: upe type: bug The issue is a confirmed bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants