[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Page MenuHomePhabricator

Add refund activity with refund reason
Closed, ResolvedPublic

Description

With gravy we are getting refund reason on the ipns:

  • make an activity like the cancel one that has that reason if its not empty (if chargeback, adyen will pass reason, add to activity)

Bonus:

  • Adyen also sends refund ipns we should check if they also send through a reason (yes, and all refund if success with reason, are chargeback, otherwise success refund will have no reason, but failed refund will have reason like no enough money in account to refund it, etc, but we do nothing but log it)

Extra Bonus:

  • Does it send who did the refund? If so add it, or can we get gravy to add this? (Slacked Gravy for this refund reason, currently do not have)

Event Timeline

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

[wikimedia/fundraising/crm@master] Add refund reason to activity for adyen

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

@Cstone Only adyen have refund reason now, but we can ask if gr4vy can pass it too, also no info from who made the refund can be found from message.

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

[wikimedia/fundraising/SmashPig@master] Log fund failed with reason for adyen

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

Change #1075985 merged by Eileen:

[wikimedia/fundraising/crm@master] Add refund reason to activity for adyen

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

Change #1076056 merged by jenkins-bot:

[wikimedia/fundraising/SmashPig@master] Log fund failed with reason for adyen

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

XenoRyet set Final Story Points to 4.