Add sherrifs as reviewers of the reverting CL |
||||||||||
Issue descriptionGiven a issue id, and a list of sheriffs, publish the CL for the sheriffs to review. API: /<issue>/publish Request: a POST request with following parameters * xsrf_token * message_only=false * add_as_reviewer=false * send_mail=true * no_redirect=true: when set to true, the response is just 'OK' upon success. * commit=false: Set as false to not commit the CL * reviewers : ',' separated email list * cc: ',' separated email list * in_reply_to: optional, use to reply to the revert in the original CL * message: the content of the message to publish Response: a string 'OK' upon success Code: https://cs.chromium.org/chromium/infra/appengine/chromium_rietveld/codereview/views.py?l=3313
,
Mar 11 2017
,
Mar 11 2017
,
Mar 13 2017
,
Mar 13 2017
,
Mar 13 2017
,
Mar 14 2017
,
Mar 14 2017
,
Mar 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/4afc175c95ef8370069a4715b893df9af7ad5758 commit 4afc175c95ef8370069a4715b893df9af7ad5758 Author: Roberto Carrillo <robertocn@google.com> Date: Tue Mar 14 22:57:38 2017 [Findit] Adding stub for rietveld's AddReviewers R=chanli,stgao BUG= 700635 Change-Id: I6ef43d2233b44672202f61940b6de52e2b356dde Reviewed-on: https://chromium-review.googlesource.com/454000 Commit-Queue: Roberto Carrillo <robertocn@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [modify] https://crrev.com/4afc175c95ef8370069a4715b893df9af7ad5758/appengine/findit/infra_api_clients/codereview/codereview.py [modify] https://crrev.com/4afc175c95ef8370069a4715b893df9af7ad5758/appengine/findit/infra_api_clients/codereview/rietveld.py
,
Mar 15 2017
,
Mar 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/c80dfe077f3d5e5cbd108e93de7471f46ee0eba8 commit c80dfe077f3d5e5cbd108e93de7471f46ee0eba8 Author: Roberto Carrillo <robertocn@google.com> Date: Wed Mar 15 23:19:13 2017 [Findit] Implement AddReviewers api for rietveld codereview. This api lets us add a list of emails as reviewers to a rietveld codereview, and optionally customize the message for the addition. R=chanli,lijeffrey,stgao BUG= 700635 Change-Id: I4852998e992a12bbe5db225cff99eebac37b82a6 Reviewed-on: https://chromium-review.googlesource.com/455472 Reviewed-by: Shuotao Gao <stgao@chromium.org> Commit-Queue: Roberto Carrillo <robertocn@chromium.org> [modify] https://crrev.com/c80dfe077f3d5e5cbd108e93de7471f46ee0eba8/appengine/findit/infra_api_clients/codereview/rietveld.py [modify] https://crrev.com/c80dfe077f3d5e5cbd108e93de7471f46ee0eba8/appengine/findit/infra_api_clients/codereview/test/rietveld_test.py
,
Mar 16 2017
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by st...@chromium.org
, Mar 11 2017