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
F15408366: D18819.id.diff
Tue, Mar 18, 10:35 PM
F15395628: D18819.id45154.diff
Sun, Mar 16, 8:33 AM
F15334904: D18819.diff
Sat, Mar 8, 1:12 PM
Unknown Object (File)
Feb 19 2025, 9:48 PM
Unknown Object (File)
Jan 30 2025, 10:32 AM
Unknown Object (File)
Jan 29 2025, 9:03 PM
Unknown Object (File)
Jan 28 2025, 2:55 AM
Unknown Object (File)
Jan 23 2025, 8:30 AM
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