Skip to content
Snippets Groups Projects

Fix webhook screen

Merged JonGold requested to merge JonGold/authnet:fix-webhook-screen into master

There's actually 2 related commits here, lmk if you want them as separate MRs.

  • The first prevents creating webhooks if the domain of the payment processor isn't the current domain. This check is present in Stripe. This was fairly serious for me because it was causing CiviRules to be triggered multiple times per contribution.
  • The second prevents automatically fixing webhooks on status check, instead implementing a workflow identical to Stripe's.

Note that the new page's path is slightly different because there's apparently a bug that if you add an action to the Status page, and the path starts with civicrm/a, it assumes it's an Angular page and screws up the URL.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading