Reviewed & tested by the community
Project:
Commerce Stripe
Version:
2.x-dev
Component:
Payment Element
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
16 Mar 2018 at 15:17 UTC
Updated:
7 Jan 2026 at 07:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
carsteng commentedHere comes the patch.
Comment #3
carsteng commentedComment #4
mglamanLet's postpone this on #2880160: Add Wero Payment Method Type, when we can finalize the best way to support these different sources with Stripe.
Comment #6
xavier.massonI'm aware about the Support iDeal through Stripe (sources) to prefer on a generic implementation of the different sources but i had recently the needs to use the SEPA debit source on a project and the quickest way to provides it was to re-use the initial work from the patch #1, so here attached the reroll of the patch with few improvements / fixes.
Comment #7
xavier.massonComment #8
a.milkovskyThe patch #6 looks good. As the related issue #2880160 haven't got any progress since 4 years, it makes sense to commit the current patch.
Comment #9
cialog commented@xavier.masson Is the SEPA payment usable with recurring payment (subscription) ?
Did someone manage to make this work for drupal 10?
Comment #10
phannphong commentedThe current patch works with rc6 but not for 8.x-1.1 version so I created a new patch here.
P/S: my mistake when the patch file is empty. Check next patch file please.
Comment #11
phannphong commentedComment #12
xavier.massonComment #13
aunvBecause the Stripe Sources API has reached end of life, I have updated the StripeSepa payment gateway to use the Payment Methods and Payment Intents APIs.
Comment #14
smokrisIf I understand correctly from #3408951: Refactor Stripe payment element for additional payment method types, the path forward for SEPA support is to implement a
CommercePaymentMethodTypeplugin; thus this issue is no longer blocked. Based on that, I updated this issue's status and summary.Comment #17
smokrisI just opened MR https://git.drupalcode.org/project/commerce_stripe/-/merge_requests/193, which makes use of the common superclass introduced on #3408951: Refactor Stripe payment element for additional payment method types. I've tested this locally with a Stripe sandbox:
The selected stripe payment method type(stripe_sepa_debit) is not currently supported..Completedstate).Comment #18
smokrisI rebased the MR, resolving the conflict by removing the update hook that was effectively identical to #3421392, under the assumption that this issue will also be part of the same tagged release as that other issue.
I performed the additional testing steps described in this comment:
charge.succeededwebhook request, and Drupal Commerce correctly updates the payment status to Completed.Auth Only with Delayed Capture and Auth+Capture— n/a; SEPA does not support delayed capture.charge.refundedwebhook request, and Drupal Commerce correctly updates the payment status to Refunded.Based on that testing, I'll tentatively move this issue along to RTBC.
Comment #19
anybodyUnified the title