Page MenuHomePhabricator

Execute Herald again after promoting revisions out of the "Draft" state
ClosedPublic

Authored by epriestley on Dec 5 2017, 3:57 PM.
Tags
None
Referenced Files
F15508246: D18819.id45154.diff
Wed, Apr 16, 3:47 AM
F15505041: D18819.id45154.diff
Mon, Apr 14, 11:28 PM
F15505039: D18819.id45159.diff
Mon, Apr 14, 11:27 PM
F15500653: D18819.id.diff
Sun, Apr 13, 6:50 PM
F15494354: D18819.diff
Sat, Apr 12, 11:23 PM
F15455527: D18819.diff
Sun, Mar 30, 2:58 AM
F15448130: D18819.diff
Fri, Mar 28, 3:17 AM
F15408366: D18819.id.diff
Mar 18 2025, 10:35 PM
Subscribers
None

Details

Summary

Fixes T13027. Ref T2543. When revisions promote from "Draft" because builds finish or no builds are configured, the status currently switches from "Draft" to "Needs Review" without re-running Herald.

This means that some rules -- notably, "Send me an email" rules -- don't fire as soon as they should.

Instead of applying this promotion in a hacky way inline, queue it and apply it normally in a second edit, after the current group finishes.

Test Plan
  • Created a revision, reviewed Herald transcripts.
  • Saw three Herald passes:
    • First pass (revision creation) triggered builds and no email.
    • Second pass (builds finished) did not trigger builds (no update) and did not trigger email (revision still a draft).
    • Third pass (after promotion out of 'draft') did not trigger builds (no update) but did trigger email (revision no longer a draft).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable