Merge lp://staging/~thekorn/launchpad-gm-scripts/show_patches into lp://staging/~ubuntu-dev/launchpad-gm-scripts/ubuntu

Proposed by Markus Korn
Status: Rejected
Rejected by: Brian Murray
Proposed branch: lp://staging/~thekorn/launchpad-gm-scripts/show_patches
Merge into: lp://staging/~ubuntu-dev/launchpad-gm-scripts/ubuntu
To merge this branch: bzr merge lp://staging/~thekorn/launchpad-gm-scripts/show_patches
Reviewer Review Type Date Requested Status
Bryce Harrington Disapprove
Brian Murray Needs Resubmitting
Review via email: mp+304@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

Could you add a screenshot of what it looks like with your change? Thanks in advance.

review: Needs Resubmitting
Revision history for this message
Bryce Harrington (bryce) wrote :

The indicator for patches is now upstream in launchpad itself, so this branch is obsolete.

Fwiw, it probably would have been better to just accept the branch when it was submitted, as the screenshot could have been done separately. It's unfortunate this branch ended up bitrotting...

review: Disapprove

Unmerged revisions

21. By Markus Korn

lp_patches.user.js: add star on top of page if a bugreport has a patch

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: