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
Unknown Object (File)
Apr 30 2025, 5:35 AM
Unknown Object (File)
Apr 23 2025, 5:15 PM
Unknown Object (File)
Apr 23 2025, 7:11 AM
Unknown Object (File)
Apr 20 2025, 11:03 AM
Unknown Object (File)
Apr 20 2025, 10:44 AM
Unknown Object (File)
Apr 18 2025, 4:55 PM
Unknown Object (File)
Apr 17 2025, 3:35 PM
Unknown Object (File)
Apr 8 2025, 2:28 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.