Merge lp:~abentley/launchpad/info-type-app-config into lp:launchpad
| Status: | Merged | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Approved by: | j.c.sackett on 2012-10-04 | ||||||||||||
| Approved revision: | no longer in the source branch. | ||||||||||||
| Merged at revision: | 16094 | ||||||||||||
| Proposed branch: | lp:~abentley/launchpad/info-type-app-config | ||||||||||||
| Merge into: | lp:launchpad | ||||||||||||
| Diff against target: | 0 lines | ||||||||||||
| To merge this branch: | bzr merge lp:~abentley/launchpad/info-type-app-config | ||||||||||||
| 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
Information type configures applications.
Description of the Change
= Summary =
Information type configures applications.
== Proposed fix ==
Answers and private projects are mutually exclusive. The Product information_type, not the license, determines the sharing policies. Embargoed sets policies to EMBARGOED_
== Pre-implementation notes ==
Discussed with deryck
== LOC Rationale ==
Part of private projects.
== Implementation details ==
Updated createProduct to use information_type to determine default sharing policies. Added Specification to the list of sharing policies.
Updated answers_usage setter to refuse LAUNCHPAD for proprietary products.
Updated _valid_
Updated Answers configuration to omit LAUNCHPAD option for proprietary products.
Updated makeProduct factory method to use an appropriate license if information_type is proprietary.
== Tests ==
bin/test test_product
== Demo and Q/A ==
Create a PROPRIETARY product. The branch, bug, specification policies should all be PROPRIETARY.
Create an EMBARGOED product. The branch and specification policies should be EMBARGOED_
Configure "support tracker". "Launchpad" should not be listed.
= Launchpad lint =
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/

This looks fine. Please remove the conflict-markers.