Merge lp:~sinzui/launchpad/commercial-project-picker into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Graham Binns |
Approved revision: | no longer in the source branch. |
Merged at revision: | 14909 |
Proposed branch: | lp:~sinzui/launchpad/commercial-project-picker |
Merge into: | lp:launchpad |
Diff against target: |
173 lines (+55/-29) 5 files modified
lib/lp/app/browser/tests/test_vocabulary.py (+26/-0) lib/lp/app/browser/vocabulary.py (+19/-1) lib/lp/registry/interfaces/product.py (+3/-0) lib/lp/registry/tests/test_commercialprojects_vocabularies.py (+6/-18) lib/lp/registry/vocabularies.py (+1/-10) |
To merge this branch: | bzr merge lp:~sinzui/launchpad/commercial-project-picker |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Graham Binns (community) | code | Approve | |
Review via email: mp+95970@code.launchpad.net |
Commit message
[r=gmb][bug=930309] Include commercial subscription in the picker details.
Description of the change
Include commercial subscription in the picker details.
Launchpad bug: https:/
Pre-
The CommercialProjects vocabulary appends parenthetical subscription
information to the project displayname. This looks very in the picker,
and can be confused with the Launchpad-Id. The information often
overruns the space for the entry and is not shown.
-------
RULES
* If the project has a commercial subscription, append it to the
picker entry details.
* This could be a performance issue.
Checking commercial_
adding the subscription information. Only projects with a commercial
Maybe the commercial subscription information can be gotten in bulk.
* Oh, commercial_
on Product.
* There are three states to present in the picker
A. commercial subscription: None
B. Commercial subscription: Active
C. Commercial subscription: Expired
QA
* Visit a bug on qastaging.
* Choose the retarget it.
* Search for a commercial project.
* Expand the some of the entries to see the details.
* Verify the entry shows commercial subscription information.
Verify the commercial project entry states that it has an active
commercial subscription.
LINT
lib/
lib/
lib/
lib/
lib/
TEST
./bin/test -vvc lp.app.
./bin/test -vvc lp.registry.
IMPLEMENTATION
Added getCommercialSu
returns None for all target types. Added getCommercialSu
the IProduct subclass to summarise the commercial subscription info.
Removed a duplicate call to getMaintainer() in the base class. Add
has_current_
was private.
lib/
lib/
lib/
Removed the overloading the of project's displayname that looked like a
Launchpad-Id and could overrun the picker overlay..
lib/
lib/
Hi Curtis,
Nice branch; r=me