Maniphest T7995

Permit Herald to remove reviewer
Closed, DuplicatePublic
Actions

Assigned To

None

Authored By

	eadler
	Apr 30 2015, 12:49 AM

Tags

Referenced Files

None

Subscribers

Description

There are certain projects (and users) which should never be a reviewer on a diff. For example, we have a project that consists of "all committers" which is useful for policy and other reasons. However, adding 300+ people to the emails and having them show up as a reviewer is something we want to avoid.

It would be nice if we could set a Herald rule "if any of X Y OR Z are reviewers" then "remove X Y and Z as reviewers".

Related Objects

Mentioned Here: T5602: Consider flags to disable project features (can tag, can have members, can be reviewer...)

Event Timeline

eadler created this task.Apr 30 2015, 12:49 AM

eadler raised the priority of this task from to Needs Triage.

eadler updated the task description. (Show Details)

eadler added a project: Herald.

eadler added a subscriber: eadler.

epriestley closed this task as a duplicate of T5602: Consider flags to disable project features (can tag, can have members, can be reviewer...) .Apr 30 2015, 12:53 AM

This doesn't seem scalable, maintaining a blocklist, that is. Is this an actual abuse case that happened?

Ah, well nevermind. haha, yeah T5602 is a slightly better solution.