| Issue 37: | Assign additional reviewers to a change through the web UI | |
| Back to list |
Reported by Shawn Pearce <sop@google.com> on Mon Jan 26 14:47:40 PST 2009 Source: JIRA GERRIT-37 Affected Version: 2.0 Allow any registered user to add one or more reviewers to an existing change through the web UI. This functionality used to exist in Gerrit1 on the publish comments page. In Gerrit2 it probably makes more sense to be in the Approvals disclosure panel, just below the reviewers table.
Sep 24, 2009
#1
code-rev...@gtempaccount.com
Sep 24, 2009
Comment by Brad Larson <bklarson@gmail.com> on Wed Feb 04 14:19:11 PST 2009
It looks like this will require a new RemoteService. Does adding the
following to PatchDetailService.java make sense?
@SignInRequired
void addReviewer(Account.Id reviewer, PatchSet.Id psid,
AsyncCallback<VoidResult> callback);
Let me know if I'm missing some already existing functionality which would
work instead.
Sep 24, 2009
Comment by Shawn Pearce <sop@google.com> on Wed Feb 04 14:20:40 PST 2009 Yup, I agree with you. Method signature looks good to me.
Sep 24, 2009
Comment by Shawn Pearce <sop@google.com> on Fri Feb 13 13:11:29 PST 2009 Fixed by http://review.source.android.com/8851
Sep 24, 2009
Update by Shawn Pearce <sop@google.com> on Fri Feb 13 13:11:29 PST 2009 Fixed in version 2.0.3.
Status:
Fixed
Sep 25, 2009
(No comment was entered for this change.)
Labels:
FixedIn-2.0.3
|
|
| ► Sign in to add a comment |