Page MenuHomePhabricator
Create Task
Maniphest T224514

Add India-specific payment methods across our pipeline
Closed, ResolvedPublic4 Estimated Story Points
Actions

Assigned To
None
Authored By
Ejegg
May 28 2019, 6:39 PM
Tags
Referenced Files
None
Subscribers
Aklapper
Cstone
Ejegg

Description

DonationInterface

  • add them to dlocal (astropay) payment_submethod config
  • add a new form definition to DonationInterfaceFormSettings.php
  • download logos where needed

SmashPig

  • add them to PaymentProviders/AstroPay/ReferenceData.php

CRM

  • add them to update_payment_instruments.php
  • add another function just like wmf_civicrm_update_7595 to wmf_civicrm.install file
  • add them to the big ol' switch block that normalizes payment instrument (wmf_civicrm.module lines 612-824)

Methods and bank codes

We're waiting for confirmation on bank transfer method PayTM Wallet (bank code PW, logo https://pay.dlocal.com/views/2.0/images/payments/PW.png).

The methods above with URLs for logos need those logos downloaded and resized to be copied into the gateway_forms/includes directory. Our standard logo height is 53 pixels tall. If the original logo is high enough resolution, it's nice to have a 2x and 3x version as well for high DPI screens (see Mastercard section of config for 2x and 3x syntax).

We can add all of them (including PayTM Wallet), just leaving PayTM Wallet commented out in DonationInterface's astropay_gateway/config/payment_submethod.yaml till we get confirmation that it's working.

Details

SubjectRepoBranchLines +/-
Update SmashPig version to 0.5.10wikimedia/fundraising/crmmaster+4 -4
Add India specific fields for dlocal.mediawiki/extensions/DonationInterfacemaster+67 -6
Adding India support with dlocal.mediawiki/extensions/DonationInterfacemaster+62 -0
Adding India support with dlocal.wikimedia/fundraising/SmashPigmaster+4 -0
Add India support with dlocal.wikimedia/fundraising/crmmaster+31 -0
Add logos for India cards & bank methodsmediawiki/extensions/DonationInterfacemaster+0 -0
Customize query in gerrit

Related Objects
Search...

StatusSubtypeAssignedTask
 ResolvedEjeggT224506 Implement India form in dlocal
 ResolvedNoneT224514 Add India-specific payment methods across our pipeline

Event Timeline

Ejegg created this task.May 28 2019, 6:39 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 28 2019, 6:39 PM
• DStrine moved this task from Triage to Next on the Fundraising-Backlog board.May 28 2019, 8:17 PM
• DStrine added a parent task: T224506: Implement India form in dlocal.
• DStrine moved this task from Next to Sprint +1 on the Fundraising-Backlog board.Jun 6 2019, 9:15 PM
• DStrine moved this task from Sprint +1 to Current Sprint on the Fundraising-Backlog board.Jun 11 2019, 7:53 PM
• DStrine added a project: Fundraising Sprint Land before Timeouts.Jun 11 2019, 8:00 PM
Ejegg added a comment.EditedJun 13 2019, 7:46 PM

(moved to task description)

Ejegg triaged this task as Medium priority.Jun 13 2019, 7:53 PM
Ejegg updated the task description. (Show Details)
Ejegg set the point value for this task to 4.
Ejegg updated the task description. (Show Details)Jun 13 2019, 7:55 PM
Ejegg updated the task description. (Show Details)Jun 14 2019, 2:52 AM
gerritbot added a comment.Jun 24 2019, 3:27 PM

Change 518746 had a related patch set uploaded (by Cstone; owner: Cstone):
[mediawiki/extensions/DonationInterface@master] Adding India support with dlocal.

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

gerritbot added a project: Patch-For-Review.Jun 24 2019, 3:27 PM
gerritbot added a comment.Jun 24 2019, 5:13 PM

Change 518762 had a related patch set uploaded (by Cstone; owner: Cstone):
[wikimedia/fundraising/SmashPig@master] Adding India support with dlocal.

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

gerritbot added a comment.Jun 24 2019, 5:56 PM

Change 518776 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[mediawiki/extensions/DonationInterface@master] Add logos for India cards & bank methods

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

gerritbot added a comment.Jun 24 2019, 6:36 PM

Change 518776 merged by jenkins-bot:
[mediawiki/extensions/DonationInterface@master] Add logos for India cards & bank methods

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

ReleaseTaggerBot added a project: MW-1.34-notes (1.34.0-wmf.11; 2019-06-26).Jun 24 2019, 7:01 PM
gerritbot added a comment.Jun 24 2019, 7:01 PM

Change 518790 had a related patch set uploaded (by Cstone; owner: Cstone):
[wikimedia/fundraising/crm@master] Add India support with dlocal.

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

gerritbot added a comment.Jun 25 2019, 1:13 AM

Change 518790 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Add India support with dlocal.

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

• DStrine added a project: Fundraising Sprint Men In Slack.Jun 25 2019, 8:44 PM
gerritbot added a comment.Jun 26 2019, 2:13 AM

Change 518762 merged by jenkins-bot:
[wikimedia/fundraising/SmashPig@master] Adding India support with dlocal.

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

Ejegg moved this task from Backlog to Doing on the Fundraising Sprint Men In Slack board.Jun 26 2019, 2:19 AM
gerritbot added a comment.Jun 26 2019, 8:15 PM

Change 518746 merged by jenkins-bot:
[mediawiki/extensions/DonationInterface@master] Adding India support with dlocal.

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

ReleaseTaggerBot edited projects, added MW-1.34-notes (1.34.0-wmf.13; 2019-07-09); removed MW-1.34-notes (1.34.0-wmf.11; 2019-06-26).Jun 26 2019, 9:00 PM
Maintenance_bot removed a project: Patch-For-Review.Jun 26 2019, 9:10 PM
Ejegg added a subscriber: Cstone.Jun 27 2019, 3:36 AM

@Cstone I deployed the DonationInterface part of this tonight. There's one more tiny thing to do - bump the version number of the SmashPig composer package and update the SmashPig library in crm. That will let the Dlocal audit code use the updated decoder ring. We should be able to put a test donation through as early as tomorrow, as long as we get that library update out before any audit file comes through with the new methods.

Ejegg added a comment.Jun 27 2019, 4:09 AM

@Cstone oops, shortly after deploying this I started seeing "Empty param x_cpf" in the error logs. That's the tax ID number we have to send along with most dlocal requests. We'll have to add that (our normalized name for the parameter is fiscal_number) to the country_fields.yaml, and add a country-specific label in the i18n files (search for fiscal_number in *.json for examples). The India-specific label for this field should be 'PAN'.

The dlocal Streamline documentation also says we need street address and city for India, so those will go in country_fields as well. And finally, we need to add an 'x_version' parameter to the newinvoice transaction definition, with fixed value 1.1

For now, I pushed out a followup patch to DonationInterfaceFormSettings dialing the form's selection_weight down to zero. I think that should make sure people don't land there just because they're trying to donate from India, in INR.

gerritbot added a comment.Jun 27 2019, 8:17 PM

Change 519510 had a related patch set uploaded (by Cstone; owner: Cstone):
[mediawiki/extensions/DonationInterface@master] Add India specfic fields for dlocal. PAN for fiscal_number and x_version.

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

gerritbot added a project: Patch-For-Review.Jun 27 2019, 8:17 PM
Cstone added a comment.Jun 27 2019, 8:17 PM

I took the definition for PAN from here https://www.incometaxindia.gov.in/pages/pan.aspx

Cstone added a comment.Jun 27 2019, 9:01 PM

@Ejegg when adding x_version . In AstroPayTest for testNewInvoiceRequest it is using BR. Should a separate test be made for IN using the x_version parameter?

Ejegg added a comment.Jun 27 2019, 9:41 PM

@Cstone let's update the existing test to add the version code. Maybe add another test for India to check that address and city come through?

gerritbot added a comment.Jul 3 2019, 7:04 PM

Change 519510 merged by jenkins-bot:
[mediawiki/extensions/DonationInterface@master] Add India specific fields for dlocal.

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

Maintenance_bot removed a project: Patch-For-Review.Jul 3 2019, 7:11 PM
Ejegg moved this task from Doing to Pending Deployment on the Fundraising Sprint Men In Slack board.Jul 6 2019, 12:38 AM
Ejegg moved this task from Pending Deployment to Doing on the Fundraising Sprint Men In Slack board.
• DStrine added a project: Fundraising Sprint Never Ending Query.Jul 9 2019, 8:36 PM
Ejegg moved this task from Backlog to Deployed on the Fundraising Sprint Never Ending Query board.Jul 10 2019, 3:38 PM
gerritbot added a comment.Jul 10 2019, 8:44 PM

Change 521952 had a related patch set uploaded (by Cstone; owner: Cstone):
[wikimedia/fundraising/crm@master] Update SmashPig version to 0.5.10

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

gerritbot added a project: Patch-For-Review.Jul 10 2019, 8:44 PM
gerritbot added a comment.Jul 10 2019, 11:35 PM

Change 521952 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Update SmashPig version to 0.5.10

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

Maintenance_bot removed a project: Patch-For-Review.Jul 11 2019, 12:10 AM
XenoRyet closed this task as Resolved.Jul 23 2019, 8:03 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits
  NODES
Note 5
Project 11