Issue 3035: Be able to remove specific scores from a reviewer while leaving the reviewer still listed on the change
Status:  Submitted
Owner: ----
Closed:  Nov 25
Project Member Reported by david.pu...@sonymobile.com, Nov 26, 2014
As discussed in the code review on [1].

We should consider moving the [x] to kill votes onto the votes themselves.

If a reviewer voted multiple time, say VRFY, CRVW & LC. Now someone
removes her VRFY vote. The VRFY is removed but the CRVW and LC stay.

We could then do more fine grained removals of votes in the ACL system. Allow removing a VRFY-1, but not a CRVW-2 for example.

We also want to be able to keep a reviewer as a reviewer on the change, but just kill their scores to work around a blocking vote. This would at present demote them to a CC status, but its better than dropping them entirely off the change where they might not get notifications anymore.

Removing a reviewer is also sensible, and doing so should also drop their votes as no longer relevant to the change.

[1] https://gerrit-review.googlesource.com/#/c/60245/

Dec 2, 2014
Project Member #1 David.Os...@gmail.com
(No comment was entered for this change.)
Status: Accepted
Dec 12, 2014
Project Member #2 David.Os...@gmail.com
https://gerrit-review.googlesource.com/62410
Status: ChangeUnderReview
Nov 25, 2015
Project Member #3 ekem...@google.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.13