Merge lp:~kirkland/launchpad/532624 into lp:launchpad
Proposed by
Deryck Hodge
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Deryck Hodge | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp:~kirkland/launchpad/532624 | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
25 lines (+2/-2) 2 files modified
lib/lp/registry/browser/person.py (+1/-1) lib/lp/registry/stories/person/xx-person-bugs.txt (+1/-1) |
||||
To merge this branch: | bzr merge lp:~kirkland/launchpad/532624 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Curtis Hovey (community) | ui | Approve | |
Deryck Hodge (community) | code | Approve | |
Review via email: mp+20775@code.launchpad.net |
Commit message
Change the person bugs page link of "show package report" to the more intuitive "list subscribed packages."
To post a comment you must log in.
The code is simple and looks good. Thanks for fixing an affected test, too. I'll run this through ec2 just to make sure all tests are covered.
You'll also need to get a UI reviewer to approve the change. Ping noodles775 or sinzui in #launchpad-reviews on Freenode for this.
Thanks again for fixing this!
Cheers,
deryck