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

Enable the use of saved SEPA payments #1471

Closed
superdav42 opened this issue Mar 31, 2021 · 3 comments · Fixed by #1540
Closed

Enable the use of saved SEPA payments #1471

superdav42 opened this issue Mar 31, 2021 · 3 comments · Fixed by #1540
Assignees

Comments

@superdav42
Copy link
Contributor

When working on #1452 I found that checking the " Save payment information to my account for future purchases." did not work for SEPA payments. I have not investigated what the issue is.
We need to allow customer to save their IBAN so they can more easily checkout in future orders.

Should probably be done after #1468

@c-shultz
Copy link
Collaborator

c-shultz commented Apr 5, 2021

It looks like this could be a good one to save for sprint 2. SEPA will work without this, right @superdav42 ?

@superdav42
Copy link
Contributor Author

Yes, it will work but we need to either disable the option or fix it before we try to launch SEPA. I'm guessing it won't take much to fix it but I haven't tried yet.

@harriswong harriswong self-assigned this Apr 6, 2021
@harriswong
Copy link
Contributor

harriswong commented Apr 7, 2021

Notes:

  • In order for the "Save payment information..." checkbox to show, make sure you are logged in as a customer (not inside wp-admin).
    image
  • If we have both "Credit Card" and "SEPA" payment methods enabled while store's currency is USD, it will fail with this error:
    image
    Change currency to EUR and it will be okay. Ideally, if the store is set to USD, "SEPA" should be hidden but I think this is something to be handled in Settings. We should test both USD/EUR when we roll this out.

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

Successfully merging a pull request may close this issue.

3 participants