Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
A
authnet
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Extensions
  • authnet
  • Merge Requests
  • !10

Merged
Opened Oct 08, 2020 by JonGold@JonGoldReporter

Fix webhook screen

  • Overview 1
  • Commits 2
  • Changes 4

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.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Reference: extensions/authnet!10
Source branch: fix-webhook-screen