Show all pending reviews for my projects

Bug #886331 reported by Jonathan Lange
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

Hello,

I maintain or am a maintainer of a fair few projects on Launchpad. It's always a wonderful thing when someone submits a patch for these projects and a correspondingly terrible thing when I miss that patch.

It would be great if there were a way of seeing all pending reviews for all of my projects on Launchpad.

Thanks,
jml

Revision history for this message
Brad Crittenden (bac) wrote :

That's a great idea Jonathan. It will probably be part of a grander dashboard initiative.

Changed in launchpad:
status: New → Triaged
importance: Undecided → Wishlist
Changed in launchpad:
importance: Wishlist → Low
tags: added: code-review projects
Revision history for this message
Jonathan Lange (jml) wrote :

To be clear, Person:+activereviews doesn't do this at the moment. It only shows reviews that you are actively engaged in. Personally I think that's a useful view, and should be preserved without clutter.

What I would like is a view, grouped by project, of the pending reviews for my projects in Launchpad. I guess that "commit access to trunk" or "member of reviewer team" are both good proxies for "my projects", although maybe a declared list would be nice.

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 886331] Re: Show all pending reviews for my projects

On Thu, Nov 10, 2011 at 12:26 AM, Jonathan Lange <email address hidden> wrote:
> To be clear, Person:+activereviews doesn't do this at the moment. It
> only shows reviews that you are actively engaged in. Personally I think
> that's a useful view, and should be preserved without clutter.

We show available reviews on project and project group pages, why
would showing them on the persons page cause clutter?

Yes, its another section, and we certainly could do a separate view,
but activereviews already shows:
 incoming specific requests
 outgoing proposals
 in-progress reviews
 proposals ready to land

Adding one section to that 'reviews that could be done', with the
stocks headings of:
Branch Merge Proposal Requested By Lines Activity

sorted by project as a prefix to any standard sort we do (which would
group everything for testtools together, for instance) - well that
seems like a small change visually, one which is (except for
ubuntu-branches) going to perform well due to the human-scale nature
of the data. Put at the bottom of the page, seems to me you wouldn't
even notice it unless you went looking for it.

This wouldn't preclude doing a more fancy implementation later of course.

Revision history for this message
Jonathan Lange (jml) wrote :

Feel free to try it that way! It's possible that it won't be too cluttered. On the other hand, it's possible that all of that other text has an impact on how it feels.

While sorting by project would be great, better still (I think) would be to group by project, each with its own heading.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.