Merge proposal review page should include a full diff

Bug #202000 reported by Daniel Schierbeck
26
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Aaron Bentley

Bug Description

When reviewing a merge proposal, one is obviously very interested in the changes made in the source branch. Currently, only the new revisions are listed. It is possible to see the diff of each revision by clicking on the revision number, but this is cumbersome, and doesn't provide the required overview.

The review page should include a complete diff.

Revision history for this message
Tim Penhey (thumper) wrote :

Yes it should.

Changed in launchpad-bazaar:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Martin Albisetti (beuno) wrote :

Please :)

Revision history for this message
Tim Penhey (thumper) wrote : Re: [Bug 202000] Re: Merge proposal review page should include a full diff

On Thursday 31 July 2008 18:02:18 Martin Albisetti wrote:
> Please :)

It's coming. RSN.

Revision history for this message
Andrew Bennetts (spiv) wrote :

Pretty please! :)

Revision history for this message
Tim Penhey (thumper) wrote :

On Thursday 11 September 2008 13:21:05 Andrew Bennetts wrote:
> Pretty please! :)

Aaron is working on it.

Revision history for this message
Tim Penhey (thumper) wrote :

It has a few dependencies, but it is coming.

Changed in launchpad-bazaar:
assignee: nobody → abentley
status: Confirmed → In Progress
Revision history for this message
Jonathan Lange (jml) wrote :

Is this the one that's being landed this week?

Changed in launchpad-bazaar:
milestone: none → 2.2.2
Revision history for this message
Aaron Bentley (abentley) wrote :

Yes. Though of course, we've had review diffs for "bzr send" reviews for ages now. This just makes sure they're everywhere.

Tim Penhey (thumper)
Changed in launchpad-bazaar:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

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