Homec4science

Move revision header warnings into custom fields

Authored by epriestley <git@epriestley.com> on Jul 2 2014, 13:58.

Description

Move revision header warnings into custom fields

Summary:
Ref T5495. We currently show one warning in revision headers, about not having any reviewers.

I want to add a second warning (for missing Legalpad signatures). At least one install would like to add custom warnings (see T5495) which are so specific that we can't reasonably cover them in the upstream.

Generalize these header warnings by moving them to CustomField, so I can implement the Legalpad stuff without making a mess and the install in T5495 can use an extension.

Test Plan:
Hit all three header states, they look exactly like they did before this change:

{F173265}

Reviewers: btrahan, chad

Reviewed By: chad

Subscribers: epriestley

Maniphest Tasks: T5495

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

Details

Committed
epriestley <git@epriestley.com>Jul 2 2014, 13:58
Pushed
aubortJan 31 2017, 17:16
Parents
rPH394dcb7900be: Fix an issue with Diviner symbol rule using incorrect logic
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH5a158b5b1932: Move revision header warnings into custom fields (authored by epriestley <git@epriestley.com>).Jul 2 2014, 13:58