Response to test webhook now includes payment processor name from CiviCRM (makes it easier to be sure which payment processor is responding)

Don't handle invoice.paid event - it gets processed at the same time as invoice.payment_succeeded and sometimes causes duplicate payments to be created

Jamie McClelland authored 4 years ago and mattwire committed 4 years ago

This merge requests also fixes the previously broken tests.

If we don't have a recurring contribution for subscription ID don't throw exception - it's probably a subscription that was recorded outside of CiviCRM so we ignore and return 200 OK to keep webhook happy

Record refund against the already recorded payment in CiviCRM so we update financial items correctly