Migrate branch visibility rules to project sharing
Bug #933938 reported by
Curtis Hovey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Steve Kowalik | ||
Bug Description
Teams with a branch visibility rule are in the project's Proprietary policy.
Related branches
lp:~stevenk/launchpad/populate-branch-aag
- William Grant: Approve (code)
-
Diff: 155 lines (+87/-1)3 files modifieddatabase/schema/security.cfg (+2/-0)
lib/lp/scripts/garbo.py (+47/-0)
lib/lp/scripts/tests/test_garbo.py (+38/-1)
lp:~stevenk/launchpad/new-branch-visibility
- William Grant: Approve (code)
-
Diff: 559 lines (+104/-116)17 files modifieddatabase/sampledata/current-dev.sql (+1/-0)
database/sampledata/current.sql (+1/-0)
database/schema/security.cfg (+1/-0)
lib/lp/code/browser/tests/test_branch.py (+1/-1)
lib/lp/code/browser/tests/test_branchmergeproposal.py (+3/-2)
lib/lp/code/browser/tests/test_product.py (+5/-4)
lib/lp/code/model/branch.py (+58/-19)
lib/lp/code/model/branchcollection.py (+7/-48)
lib/lp/code/model/tests/test_branchcollection.py (+0/-14)
lib/lp/code/model/tests/test_branchmergeproposal.py (+7/-7)
lib/lp/code/model/tests/test_branchsubscription.py (+1/-1)
lib/lp/code/stories/branches/xx-branch-listings.txt (+7/-6)
lib/lp/code/stories/feeds/xx-branch-atom.txt (+1/-2)
lib/lp/code/tests/test_branch_access_policy_triggers.py (+4/-3)
lib/lp/registry/browser/tests/test_pillar_sharing.py (+3/-4)
lib/lp/registry/stories/product/xx-product-development-focus.txt (+2/-3)
lib/lp/security.py (+2/-2)
lp:~wgrant/launchpad/bug-933938-mps
- Curtis Hovey (community): Approve (code)
-
Diff: 128 lines (+11/-72)1 file modifiedlib/lp/code/model/branchcollection.py (+11/-72)
| tags: | added: sharing |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| Changed in launchpad: | |
| assignee: | nobody → Steve Kowalik (stevenk) |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Steve Kowalik (stevenk) wrote | #3 |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
r15532 in stable (http://