Add the ability to request review from any of the suggested reviewers in one action

NEW
Unassigned

Status

()

bugzilla.mozilla.org
Extensions: Review
--
enhancement
5 years ago
3 years ago

People

(Reporter: ted, Unassigned)

Tracking

({bmo-big})

Production
bmo-big

Details

(Reporter)

Description

5 years ago
Often times people don't care who reviews their patch, simply that it gets review from a peer of the module in question. It would be nice if Bugzilla could facilitate this, by allowing you to request review from "a peer of module X" in some way. My ideal implementation of this would function something like:
1) User requests review from "peers@moduleX"
2) The review request shows up in review queries for all users in the group peers@moduleX, sends flag mail to all users in peers@moduleX.
3) When someone clears the r?peers@moduleX flag, the review is removed from review queries for all users in peers@moduleX.

dkl noted on IRC that a first approximation of this would be something like "expand r?peers@moduleX -> r?peer1 r?peer2 r?peer3 ...". This would be a decent half-step, but it'd require manually clearing the other requests after reviewing, and also make it unclear whether multiple reviews were desired or just a single review from any requestee.
Probably best to work on this as an extension of our current Review extension on BMO and not necessarily upstream.

dkl
Assignee: attach-and-request → nobody
Component: Attachments & Requests → Extensions: Review
Product: Bugzilla → bugzilla.mozilla.org
QA Contact: default-qa
Version: unspecified → Development/Staging
by "peer" i'm going to assume you mean "suggested reviewers".  while strictly speaking these two aren't the same, we already a list of suggested reviewers and i think it's close enough to suit the requirements here.

> dkl noted on IRC that a first approximation of this would be something like
> "expand r?peers@moduleX -> r?peer1 r?peer2 r?peer3 ..."

i'm concerned that would cause more trouble than it fixes, and would like to echo ted's concerns surrounding the multiple review case.
Keywords: bmo-big
Summary: Add the ability to request review from any of a group of peers in one action → Add the ability to request review from any of the suggested reviewers in one action
Version: Development/Staging → Production
(Reporter)

Comment 3

4 years ago
Suggested reviewers would be fine.
You need to log in before you can comment on or make changes to this bug.