Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
We can support saved payment methods by implementing the Customer and Customer Card APIs
Comment | File | Size | Author |
---|---|---|---|
#18 | 2869137-18.patch | 10.53 KB | oheller |
#16 | 2869137-16.patch | 10.75 KB | oheller |
#14 | commerce_square-reusable_payments-2869137-14.patch | 44.46 KB | focal55 |
#7 | commerce_square-reusable_payments-2869137-7-D8.patch | 10.08 KB | nnevill |
Comments
Comment #2
focal55 CreditAttribution: focal55 commentedI took a pass at implementing this. It provides a checkbox on the payment method add form allowing the user to opt in for storing the card on file. If the user opts in, the customer and card records is store at Square.
Comment #3
focal55 CreditAttribution: focal55 commentedI fixed up the coding standard errors in this patch.
Comment #4
focal55 CreditAttribution: focal55 commentedI used the AddressItem class to set the address data in this patch.
Comment #5
mglamanThanks! I'll review. One of the blockers was to get this optional select in 2.0 core
Comment #7
nnevillHere is working versions for reusable square payments for 8.x-1.2 version.
Comment #8
vuilIs this issue related? #3069064
My Square Connect module doesn't work at all.
Comment #9
vuilI also update #7 patch with the mentioned issue [#3069064] to the latest 8.x-1.x-dev version.
Comment #10
vuilComment #12
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedthis no longer applies after the new dev version.
Comment #13
mglamanHiding the patch from ilchovuchkov in #9. It's a mismash.
#7 will need a reroll against fixes for the 2019-03-15 API changes going into -dev.
I like this approach until Commerce itself adds this checkbox.
Comment #14
focal55 CreditAttribution: focal55 commentedThis seems to work for me against the lastest version of dev.
Comment #15
oheller CreditAttribution: oheller at Electric Citizen commentedThis patch is using tabs for indents instead of two spaces. It is creating havoc when trying to implement further patches on these files.
Comment #16
oheller CreditAttribution: oheller at Electric Citizen commentedI've rerolled the patch 14.
Comment #17
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedno longer applies to 1.4 version
Comment #18
oheller CreditAttribution: oheller at Electric Citizen commentedHere is a rerolled patch for 1.4.
Comment #19
oheller CreditAttribution: oheller at Electric Citizen commentedComment #21
vmarchukWe should take into account this issue #2871483 as it's described here
Comment #22
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedWhy are there two reusable checkboxes
Comment #23
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedEdit: I no longer have the issue below.
It seems to work for me.
Slight modification would be to remove the redundant code for the checkbox.
I apply the patch
select recurring
the initial order is settled as expected
The recurring order is settled also.
----
I apply the patch
I can go through the checkout just fine as anonymous user.
If I am logged in there is an error on payment.
I get the following:
We encountered an unexpected error processing your payment method. Please try again later.
and then in my logs:
mysite/checkout/80/review
Message [HTTP/2 400] @"errors":[{"category":"INVALID_REQUEST_ERROR","code":"MISSING_REQUIRED_PARAMETER","detail":"Missing required parameter.","field":"customer_id"]}
Comment #24
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedComment #25
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedThis no longer applies to the module.
Comment #26
as_vardAny news?
Comment #27
vuil