Merge lp:~gary/launchpad/bug770287 into lp:launchpad
Proposed by
Gary Poster
Status: | Merged |
---|---|
Approved by: | Gary Poster |
Approved revision: | no longer in the source branch. |
Merged at revision: | 12919 |
Proposed branch: | lp:~gary/launchpad/bug770287 |
Merge into: | lp:launchpad |
Diff against target: |
126 lines (+33/-17) 2 files modified
lib/lp/bugs/browser/structuralsubscription.py (+9/-10) lib/lp/registry/browser/tests/test_subscription_links.py (+24/-7) |
To merge this branch: | bzr merge lp:~gary/launchpad/bug770287 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Benji York (community) | code | Approve | |
Review via email: mp+58979@code.launchpad.net |
Commit message
[r=benji][bug=770287] If there is no bugtracker on a product, do not render links, even if the parent project has a bugtracker
Description of the change
This branch fixes an edge case, as described in https:/
Using the helper seemed the most elegant approach: the helper's "pillar" was exactly the value we wanted.
lint is happy, after rearranging some comments.
Thank you.
To post a comment you must log in.
Looks good.