Launchpad itself

Page that shows reviews I *must* do and reviews I *can* do

Bug #325985 reported by Jonathan Lange
24
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Paul Hummer
Launchpad itself 2.2.6

Bug Description

I like doing code reviews so much that I don't want to waste my time looking for reviews to do -- I want Launchpad to tell me.

Launchpad should have a page linked to from my overview that shows me, in two separate lists:
    1. reviews I must do
    2. reviews I could do

The first would be a list of reviews that I have been specifically requested to do. It should also include merge proposals that I've already reviewed, marked as needs-fixing or whatever and had someone reply to.

The second would be a list of reviews that a team I'm in has been requested to review.

Jonathan Lange (jml)
Changed in launchpad-bazaar:
importance: Undecided → Medium
status: New → Triaged
Tim Penhey (thumper)
Changed in launchpad-bazaar:
assignee: nobody → thumper
milestone: none → 2.2.2
status: Triaged → In Progress
Revision history for this message
Tim Penhey (thumper) wrote :

Fixed in RF 7780.

Changed in launchpad-bazaar:
status: In Progress → Fix Committed
Revision history for this message
Jonathan Lange (jml) wrote :

Actually, RF 7780 fixes bug 325987.

There's still no per-user page that shows all reviews that need to be done across all projects.

Changed in launchpad-bazaar:
milestone: 2.2.2 → none
status: Fix Committed → Triaged
Revision history for this message
Paul Hummer (rockstar) wrote :

I don't mean to yoink this from Tim, but I just found out there's a bug for this, and so I thought I'd note that I'm working on the per-user reviews. I'll link it when it's actually fixing the bug.

Changed in launchpad-code:
assignee: Tim Penhey (thumper) → Paul Hummer (rockstar)
status: Triaged → In Progress
Revision history for this message
Martin Pool (mbp) wrote : Re: [Bug 325985] [NEW] Page that shows reviews I *must* do and reviews I *can* do

2009/2/6 Jonathan Lange <email address hidden>:
> Public bug reported:
>
> I like doing code reviews so much that I don't want to waste my time
> looking for reviews to do -- I want Launchpad to tell me.
>
> Launchpad should have a page linked to from my overview that shows me, in two separate lists:
>    1. reviews I must do
>    2. reviews I could do
>
> The first would be a list of reviews that I have been specifically
> requested to do. It should also include merge proposals that I've
> already reviewed, marked as needs-fixing or whatever and had someone
> reply to.
>
> The second would be a list of reviews that a team I'm in has been
> requested to review.

This is a very good idea, but it should really say in the UI (maybe in
a popup help or just in smaller print) what those two headings mean.
The text jml has above is fine.

--
Martin <http://launchpad.net/~mbp/>

Revision history for this message
Martin Pool (mbp) wrote :

It's looking really nice, though I think it does still need explanation text.

Paul Hummer (rockstar)
Changed in launchpad-code:
importance: Medium → High
Revision history for this message
Paul Hummer (rockstar) wrote :

I should also mention this was in devel r8635

Changed in launchpad-code:
milestone: none → 2.2.6
status: In Progress → Fix Committed
Paul Hummer (rockstar)
Changed in launchpad-code:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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