Page MenuHomePhabricator

Prevent "Call webhooks" Herald action from appearing in UI for adapters which can't fire it
ClosedPublic

Authored by epriestley on Feb 11 2018, 1:59 PM.
Tags
None
Referenced Files
F19153758: D19061.diff
Thu, Dec 11, 12:02 PM
F19153756: D19061.diff
Thu, Dec 11, 12:02 PM
F18906134: D19061.id.diff
Nov 8 2025, 5:10 PM
F18893120: D19061.diff
Nov 7 2025, 12:36 PM
F18852494: D19061.diff
Oct 31 2025, 9:28 AM
F18816984: D19061.id45695.diff
Oct 21 2025, 11:07 AM
F18773542: D19061.id.diff
Oct 9 2025, 10:36 AM
F18773541: D19061.id45697.diff
Oct 9 2025, 10:36 AM
Subscribers
None

Details

Summary

See https://discourse.phabricator-community.org/t/herald-webhook-is-not-called-but-herald-transcript-tells-me-the-opposite/1098.

The "Commit Hook" events don't operate on objects and don't use TransactionEditors. They can't call webhooks in a normal way and currently don't call webhooks at all. Stop offering these actions in the UI.

The "Outbound Mail" event also fires oddly and likely doesn't make much sense to hook anyway.

Test Plan

Verified that these events no longer offer "Call webhooks", while normal events still do.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 11 2018, 2:15 PM
This revision was automatically updated to reflect the committed changes.