Page MenuHomePhabricator

Support DLocal recurring cards in DonationInterface
Closed, ResolvedPublic8 Estimated Story Points

Description

  • add the recurring flag to the createPayment API call
  • make sure the returned token gets sent to the donations queue
  • include a test that looks at the queue message

For BRAZIL: special logic to show donors that recurring is not supported on debit cards. Either look for MD / VD bank code, or any card which has 3DS turned on.

Event Timeline

Ejegg added a subscriber: EMartin.

@EMartin any suggestions for the UI / messsaging when a Brazil donor who wants to make a recurring payment enters a debit card? We will find that out AFTER we have already authorized the one-time payment.

How about: Thank you for your one time donation. Due to bank regulations, recurring donations are only possible on credit cards at this time. Please enter a credit card to setup a recurring donation. Thank you.

Dwisehaupt set the point value for this task to 8.Feb 6 2023, 9:22 PM

Dropping this back in backlog because there's too much in progress in DonationInterface to touch the code without creating more merge conflicts

Change 893274 had a related patch set uploaded (by AndyRussG; author: AndyRussG):

[wikimedia/fundraising/SmashPig@master] Set processor_contact_id for recurring

https://gerrit.wikimedia.org/r/893274

Change 893275 had a related patch set uploaded (by AndyRussG; author: AndyRussG):

[mediawiki/extensions/DonationInterface@master] dLocal: Add response data for recurring

https://gerrit.wikimedia.org/r/893275

Attached patches add the required fields to the donations queue message. Test and UI elements will be included in follow-on patches.

Change 893274 abandoned by AndyRussG:

[wikimedia/fundraising/SmashPig@master] dLocal: set processor_contact_id for recurring

Reason:

processor_contact_id not needed!

https://gerrit.wikimedia.org/r/893274

Change 894692 had a related patch set uploaded (by Ejegg; author: Ejegg):

[mediawiki/extensions/DonationInterface@master] Add card type warning for recurring Brazil donors

https://gerrit.wikimedia.org/r/894692

Change 893275 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] dLocal: Add response data for recurring

https://gerrit.wikimedia.org/r/893275

Change 894692 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Add card type warning for recurring Brazil donors

https://gerrit.wikimedia.org/r/894692

Change 893413 had a related patch set uploaded (by Wfan; author: Jgleeson):

[wikimedia/fundraising/dev@master] Add dLocal cc recurring links to main page

https://gerrit.wikimedia.org/r/893413

Change 893413 merged by Wfan:

[wikimedia/fundraising/dev@master] Add dLocal cc recurring links to main page

https://gerrit.wikimedia.org/r/893413

So all we're lacking here is the logic to detect when a non-recurrable card has been used and avoid sending the recurring payment token in that case, so the donor gets the TY letter with 'one time' text.

Change 897901 had a related patch set uploaded (by AndyRussG; author: AndyRussG):

[mediawiki/extensions/DonationInterface@master] Move PlaceholderFiscalNumber to gateway_common

https://gerrit.wikimedia.org/r/897901

XenoRyet set Final Story Points to 4.

Change 897901 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Move PlaceholderFiscalNumber to gateway_common

https://gerrit.wikimedia.org/r/897901

Change 900717 had a related patch set uploaded (by Wfan; author: Wfan):

[mediawiki/extensions/DonationInterface@master] Move PlaceholderFiscalNumber to dlocal_gateway

https://gerrit.wikimedia.org/r/900717

Change 900717 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Move PlaceholderFiscalNumber to dlocal_gateway

https://gerrit.wikimedia.org/r/900717