cubicweb-vcreview #2533303 Make reviewer list attached to a project configurable [open]

For now, we have one hardcoded "reviewers" group, which cannot be "bypassed" on a given project.

We should manage reviewer groups without any hardcoded relation/constraint. In my point of view:

  • there should be one default "reviewers" CWGroup,
  • there should be a "reviewed_by" (or similar) relation between a reviewed project and a CWGroup,
  • upon creation of a project (or upon configuration of a project for it being reviewed), a hook may create a relation between the project and the default reviewers group,
  • this relation can be later modified by the project manager
  • there should also be a "reviewed_by" relation between the reviewed project an a CWUser (so one can add a given user as reviewer for a project).
priorityminor
typeenhancement
done in<not specified>
load1.000
load left1.000
closed by<not specified>

Workflow history

from state (2)to statecommentdateUser
waiting feedbackopen

No it's not

2013/09/26 16:45 UTCddouard
openwaiting feedback2013/09/25 12:25 UTCjcristau

Comments

  • 2012/12/06 10:28, written by pydavid

    Sound sensibles to me.

  • 2013/09/25 12:25, written by jcristau

    isn't this pretty much what http://www.cubicweb.org/2729751 implemented?

    you can select on a per-repository (not project, but close enough) basis whether to use the global "reviewers" group or whether to use the repository_reviewer relation.

add comment