Page MenuHomePhabricator

Add ability to accept JSON notifications
Closed, ResolvedPublic4 Estimated Story Points

Description

The new integration has JSON messages available instead of SOAP.

https://docs.adyen.com/development-resources/webhooks

We use these to get information on new iDeal recurrings as well as in a situation where a donor was never redirected back to our site.

Event Timeline

Ejegg set the point value for this task to 4.
Ejegg subscribed.

Copying estimate over from duplicate task

Change #1017340 had a related patch set uploaded (by Damilare Adedoyin; author: Damilare Adedoyin):

[wikimedia/fundraising/SmashPig@master] Add tests for JSON and Soap Authorisation messages. Modified the AdyenRestListener class similar to the soap listener to get the tests to pass.

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

Change #1017340 merged by jenkins-bot:

[wikimedia/fundraising/SmashPig@master] Add tests for JSON and Soap Authorisation messages. Modified the AdyenRestListener class similar to the soap listener to get the tests to pass.

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

The associated patch is merged. Is there more to review?

@Ejegg I don't think there's any left. What's left is to deploy and test the json notifications.

Ejegg moved this task from Backlog to Done on the Fundraising Sprint: ick() board.

We have switched over the production IPNs to JSON!