Launchpad reviews (was Re: Patch Pilot report)

[Martin Pool]

I'm sorry this went so far off the rails.  It was starting out as a
bit of a general-idea-gathering thread from our use of reviews so far,
but turned into something sounding too much like unconstructive
complaints.  I know Aaron and Tim really are working hard on improving
what is already a very useful tool, and making good progress.

In this thread and the off-list conversation we did identify bugs and
areas for development.  It is better to have them in bug reports
rather than mail though, so now here they are -- some old, some new:

The source of the confusion about team vs individual reviews, is
<https://bugs.edge.launchpad.net/launchpad-code/+bug/488231>,
confusion of "" vs None causing reviews not to be automatically
claimed when they should be.

<https://bugs.edge.launchpad.net/launchpad-code/+bug/486989> no link
from a project's branch list page to its active code reviews.

<https://bugs.edge.launchpad.net/launchpad-code/+bug/487893> no link
from a mp page to the set of all mps

<https://bugs.edge.launchpad.net/launchpad-code/+bug/487424> no way to
delete a requested review

<https://bugs.edge.launchpad.net/launchpad-code/+bug/487882> ajax
review form doesn't update the headers

<https://bugs.edge.launchpad.net/launchpad-code/+bug/488537> review
list page headings are unclear/undocumented

<https://bugs.edge.launchpad.net/launchpad-code/+bug/488540> may want
an assignee for reviews?

<https://bugs.edge.launchpad.net/launchpad-code/+bug/320068> set mp
status at the same time as doing a review

<https://bugs.edge.launchpad.net/launchpad-code/+bug/373078> no clear
code review status for 'merge with some changes'

<https://bugs.edge.launchpad.net/launchpad-code/+bug/488544> unclear
meaning of resubmit action

Hope that helps,
-- 
Martin <http://launchpad.net/~mbp/>