1275: Add capabilites for tracking reviews in progress, and for marking that a review has been read and reacted to

risk****@gmai***** (Google Code) (Is this you? Claim this profile.)
March 21, 2011
774
Describe the enhancement and the motivation for it.

We've been reviewing an extensive part of code (64k LOC of C++ code) using
the review-board and found two things missing.

1 - it is not possible to see which diffs are currently being reviewed by
someone

Suggestion: Add a new column in the dashboard with "reviews in progress",
so that if someone has started a review (but not published it yet) others
could see that.

2 - it is not possible to see who has read and reacted to the reviews

Suggestion: it should be able to mark a review "read and reacted to" so
that it's "closed" in some way. If someone then later adds new comments to
an already "read and reacted to" review it becomes "open" again or
something similar.

Basically it was hard to distribute the reviews to developers (avoid
duplicate reviews happening concurrently), and it was hard to then
afterwards distribute the revewing of the reviews back to the original
group of developers.

What operating system are you using? What browser?
- windows 7 and linux, firefox

Otherwise, great tool. Thanks!

espen
#1 olivier.*********@gmai***** (Google Code) (Is this you? Claim this profile.)
I strongly agree there is a need for an "in progress" feature.
david
#2 david
  • +Component-Reviews
david
#3 david
I think we're going to get the first part of this with bug 774 and the second part of this with bug 1277.