Closed Bug 1313654 Opened 8 years ago Closed 6 years ago

Draft page should show "r+" and "r?" on each reviewer name.

Categories

(MozReview Graveyard :: General, defect)

Product:
MozReview Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: arai, Unassigned)

Details

when I push a changeset with multiple reviewers to review repository, draft page shows reviewer list and single current status ("r?" or "r+") for each changeset.
but apparently it internally holds separated review status for each reviewer.

so, in the following situation it's confusing:
  * there are 2 reviewers (A and B) for single changeset
  * A r+ed for previous revision, on mozreview
  * B hasn't r+ed on mozreview (e.g. r+ed on bugzilla or somewhere)
the draft page just shows "r+" for the changeset, with reviewers "A,B".
it looks like nothing happens for the changeset when I publish.
but actually, B will receive review request.

so, reviewers display in the draft page should be the same as published page, that shows "r?" and "r+" for each reviewer.
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.