Page MenuHomePhabricator

Index "Call Webhook" in Herald, and show calling rules on the Webhook page

Authored by epriestley on Mar 7 2019, 2:37 PM.



Depends on D20259. Now that we can index Herald rules to affected objects, show callers on the "Webhooks" UI.

A few other rule types could get indexes too ("Sign Legalpad Documents", "Add Reviewers", "Add Subscribers"), but I think they're less likely to be useful since those triggers are usually more obvious (the transaction timeline makes it clearer what happened/why). We could revisit this in the future now that it's a possibility.

Test Plan

Screen Shot 2019-03-07 at 6.34.26 AM.png (444×698 px, 32 KB)

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

  • Push fixes for null/empty behavior back to the previous revision.
amckinley added inline comments.


This revision is now accepted and ready to land.Mar 7 2019, 10:12 PM

Oh, I'll stick like a setLimit(10) here and on the similar one in BuildPlan before I land this. I doubt anyone is going to write 19,000 herald rules to run a particular plan or call a particular hook, but no sense in breaking the page if they get super creative.


This is a list of Rules which have a "Call Webhook: ..." action -- they may not have actually fired yet.

Although you're probably using this panel to answer a question like "Which rule is calling / just called this hook?", it could also answer a question like "I'm pretty sure I wrote a rule for this but the hook isn't being called, which rule did I write?".

  • Add the limits to prevent crazy behavior if you write a million rules to call a single hook/plan.
Owners added a subscriber: Restricted Owners Package.Mar 7 2019, 10:20 PM
This revision was automatically updated to reflect the committed changes.