Page MenuHomePhabricator

dLocal missing tax ID + status mismatch errors
Closed, ResolvedPublic

Description

@LWadleigh noticed that a Gravy/Dlocal donor got rejected for 'missing tax id' even though they're in Chile where we no longer ask for the ID in the form. Looking at Gravy there are lots of recent rejections for this reason from Chile, Colombia and Uruguay (filter on status failed, connection dLocal) - all countries where we no longer ask for the ID.

Recent examples from Gravy are not populating at dLocal's console, but going back to the 6th, 229708602.5 has a status mismatch, it is Paid at dLocal, Failed at Gravy. cid=63410732 does not show the donation from the 6th. From Gravy's console it looks like this is affecting hundreds of donors per day.

Status mismatch possibly similar to T400479: Missing contributions in civicrm due to Gravy - Dlocal Transaction status mismatch

Slack thread, here we have the update from gravy that they will remove tax ID requirements for CL CO and UY on dlocal-card .

So once that request done, we can reapply the tax_id waiver

Event Timeline

@MBeat33, we removed the fiscal number / tax ID requirements for these countries after dLocal advised here that they were no longer needed, but it appears that the updates might not have made it to their API!

We're gonna roll back the removal of the fields for now, to prevent any further failures and give us time to reach out to dLocal to see what's gone wrong here.

That change has been deployed, and I can now see successful donations from Chilean donors coming in here

XenoRyet set Final Story Points to 2.
AnnWF triaged this task as Medium priority.
AnnWF updated the task description. (Show Details)
AnnWF updated the task description. (Show Details)

since gravy did not update waiver on their end, so failed, and before they fix that we have the waiver reverted for now https://gerrit.wikimedia.org/r/c/mediawiki/extensions/DonationInterface/+/1176544, and once gravy fixed this, we will revert it back~

for status mismatch dlocal is working on that, due to they send two ipn message to gravy, first reject second capture, and the ticket id is #27037683

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

[mediawiki/extensions/DonationInterface@master] Dlocal is fine with no payer.document for waiver countries

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

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

[wikimedia/fundraising/dev@master] Remove tax id for dlocal CO CL and UY from url

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

AnnWF moved this task from Done to Pending Deployment on the Fundraising Tech - Chaos Crew board.
AnnWF added a subscriber: jgleeson.

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

[wikimedia/fundraising/SmashPig@master] do not force fiscal number if not ar br for dlocal

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

Gravy confirmed today that they add the wavier for dlocal in their system. ready to remove tax_id from gravy dlocal

Change #1180632 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Dlocal is fine with no payer.document for waiver countries

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

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

[mediawiki/extensions/DonationInterface@master] Gr4vy add waier to dlocal card

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

Change #1180640 merged by Wfan:

[wikimedia/fundraising/dev@master] Remove tax id for dlocal CO CL and UY from url

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

Change #1182966 merged by jenkins-bot:

[mediawiki/extensions/DonationInterface@master] Gr4vy add waier to dlocal card

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

Change #1180915 merged by jenkins-bot:

[wikimedia/fundraising/SmashPig@master] do not force fiscal number if not ar br for dlocal

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

AKanji-WMF changed Final Story Points from 2 to 4.