Merge lp:~sinzui/launchpad/do-not-release into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Edwin Grubbs |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~sinzui/launchpad/do-not-release |
Merge into: | lp:launchpad |
Diff against target: |
167 lines (+93/-6) 4 files modified
lib/lp/registry/browser/product.py (+17/-3) lib/lp/registry/browser/tests/product-portlet-packages-view.txt (+74/-1) lib/lp/registry/stories/product/xx-product-index.txt (+0/-1) lib/lp/registry/templates/product-portlet-packages.pt (+2/-1) |
To merge this branch: | bzr merge lp:~sinzui/launchpad/do-not-release |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Edwin Grubbs (community) | code | Approve | |
Review via email: mp+22239@code.launchpad.net |
Description of the change
This is my branch to not release suggest project packages portlet.
lp:~sinzui/launchpad/do-not-release
Diff size:
Launchpad bug: https:/
Test command: ./bin/test -vv \
-t product-
-t xx-product-index
Pre-
Target release: 10.03
not release suggest project packages portlet
-------
The portlet that suggests packages that a project provides should not be
visible to general users because it is not yet possible to say that the
project does not provide any packages.
Rules
-----
* The portlet should only be visible on dev, staging and edge.
* Bonus: Do not show obsolete packages and limit the number to the
5 most recent.
QA
--
* Visit /gdp on edge and staging and verify the packages portlet
suggests packages
* Visit lp.net and verify /gdp does not suggest packages.
* Visit /bzr and verify that every package has an upload time and that
there are fix of them.
Lint
----
Linting changed files:
lib/lp/
lib/lp/
lib/lp/
lib/lp/
Test
----
* lib/lp/
* Added a test for sourcepackages and can_show_portlet
* lib/lp/
* Updated the test to verify that obsolete packages are not shown.
Implementation
--------------
* lib/lp/
* Added a property to test if the portlet should be shown.
This property will continue to be used after development. In the
future it will replace the lpnet check with a check for a confirmation
that the project is already known not to be packaged.
* Add a property to filter the obsolete sourcepackages and limit them
to 5.
* lib/lp/
* Updated the template to use the two properties.
Looks good.