#3071 ability to ack patch attachments to trac tickets
Closed: Fixed Opened by rmeggins.

The 389 is moving from using bugzilla.redhat.com as its upstream bug tracking system to using the new https://fedorahosted.org/389 trac instead. One really great feature of bugzilla that appears to be missing in trac is the ability to ack+ patch attachments. In our development work flow, in bugzilla, we attach patches to bugs, then request a review. The reviewer can simply rmeggins+ (to approve) or rmeggins- (to nack) or rmeggins? (to request more info) each individual patch attachment. Can we have such a feature in the https://fedorahosted.org/389 trac?


Some possible options:

We have ReviewBoard: https://fedorahosted.org/reviewboard/dashboard/ but it's a bit slow and might be more than you are looking for.

Also, you can customize the workflow in trac:
http://trac.edgewall.org/wiki/TracWorkflow
See the section about "optional generic review state". Would that work?

Thanks. And I also found the CustomFields plugin. These 3 options should suffice.

ok. Also, we have just added the mastertickets plugin, which adds blocks and blockedby. You may be able to use that to have a 'review' tracker ticket that blocks new tickets until they are reviewed.

If there's anything you need from us, feel free to re-open this and let us know.

Metadata