HomePhabricator

Audit / Herald - make audits added via herald "required"

Description

Audit / Herald - make audits added via herald "required"

Summary: as opposed to "requested". Also re-jigger how the "reason" works so the herald editor can get more specific data rather than a generic message. Fixes T6345 along with companion diff D10726.

Test Plan: made a herald rule to add auditors to a commit and saw it work!

Reviewers: epriestley, chad

Reviewed By: chad

Subscribers: Korvin, epriestley

Maniphest Tasks: T6345

Differential Revision: https://secure.phabricator.com/D10730

Details

Provenance
btrahanAuthored on
btrahanPushed on Oct 20 2014, 11:46 PM
Reviewer
chad
Differential Revision
D10730: Audit / Herald - make audits added via herald "required"
Parents
rPeaf6ca3b6433: Normalize AuditStatusConstant Colors
Branches
Unknown
Tags
Unknown
Tasks
T6345: No bar color for commits on Dashboard