Implement doPayment instead of doTransferCheckout #114

artfulrobot / uk.artfulrobot.civicrm.gocardless

A CiviCRM extension providing GoCardless integration to handle UK Direct Debits.

GNU Affero General Public License v3.0

5 stars 18 forks source link

Closed artfulrobot closed 3 years ago

artfulrobot commented 3 years ago

I believe this fixes #114 Tests don't cover this, but they do still pass :-)

Implement doPayment and remove doTransferCheckout (and doDirectPayment)
change name of an internal method to something more obvious (doTransferCheckoutWorker → createRedirectFlowURL)

Does it look ok to you, @mattwire ?

artfulrobot commented 3 years ago

@mattwire think I've accidentally merged this, but if you do get a chance to glance at it before the next release that would be good.

mattwire commented 3 years ago

@artfulrobot Looks good to me!

artfulrobot commented 3 years ago

Thanks @mattwire