Merge lp:~sinzui/launchpad/external-bug-tracker into lp:launchpad
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Curtis Hovey on 2012-10-04 | ||||
| Approved revision: | no longer in the source branch. | ||||
| Merged at revision: | 16101 | ||||
| Proposed branch: | lp:~sinzui/launchpad/external-bug-tracker | ||||
| Merge into: | lp:launchpad | ||||
| Diff against target: | 0 lines | ||||
| To merge this branch: | bzr merge lp:~sinzui/launchpad/external-bug-tracker | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| j.c.sackett (community) | 2012-10-04 | Approve on 2012-10-04 | |
|
Review via email:
|
|||
Commit Message
Do not show bug listings on projects that do not use Lp Bug Tracking.
Description of the Change
The bug-column feature that changes how bug listing were made lost the
rules to not show bug listings. The view ignores the project configuration,
showing bugs when the project does not use Lp to track bugs.
-------
RULES
Pre-
* The template uses not: view/should_
show the external or bug configuration information, but it
never uses just view/should_
the search form and listings.
QA
* Visit https:/
* Verify there is no search form or advanced search link.
* View the source and verify that there is no bug-table-listing.
LINT
lib/
lib/
LoC
I have a credit of 3000+ lines this week.
TEST
./bin/test -vvc lp.bugs.
IMPLEMENTATION
Updated the template to wrap the search form and listings with a
conditional view/should_
lib/
lib/

Looks good, other than one typo. I'll trust you to fix it after approval.
#9: makePerson(), not makePerson