Allow Registry Admins to change pillar and person names
Bug #588773 reported by
Curtis Hovey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Brad Crittenden | ||
Bug Description
25% of all questions asked in /launchpad-project require a pillar or person name change. Some are just about name changes, others requests require the name change before another operation can be completed, like registrting a project group. Only admins are permitted to do this, and it make take week for the admins to complete the task. In most, if not all, cases the admin is relying on the CHR engineer to agree the name should be changed.
Allowing the Registry Admins to fix names will reduce the time from days to minutes for many Answers/CHR requests. This will let the Admins work on tasks that are in their domain.
Related branches
lp:~bac/launchpad/bug-588773
- Aaron Bentley (community): Approve (code)
-
Diff: 1369 lines (+286/-17)3 files modifiedlib/canonical/launchpad/security.py (+186/-11)
lib/lp/registry/browser/tests/test_edit_permissions.py (+97/-0)
lib/lp/registry/stories/product/xx-product-edit.txt (+3/-6)
lp:~bac/launchpad/bug-588773-take2
- Curtis Hovey (community): Approve (code)
-
Diff: 1254 lines (+280/-156)35 files modifiedlib/canonical/launchpad/permissions.zcml (+0/-9)
lib/canonical/launchpad/security.py (+21/-17)
lib/canonical/launchpad/webapp/tests/test_login.py (+3/-3)
lib/lp/answers/browser/configure.zcml (+3/-3)
lib/lp/answers/browser/faqcollection.py (+1/-1)
lib/lp/answers/browser/tests/faq-views.txt (+2/-2)
lib/lp/answers/configure.zcml (+1/-1)
lib/lp/answers/doc/faqtarget.txt (+13/-12)
lib/lp/answers/doc/workflow.txt (+1/-1)
lib/lp/answers/templates/faq-listing.pt (+6/-4)
lib/lp/answers/templates/faqcollection-portlet-faqs.pt (+1/-2)
lib/lp/answers/templates/question-linkfaq.pt (+1/-1)
lib/lp/registry/browser/configure.zcml (+7/-7)
lib/lp/registry/browser/distribution.py (+1/-2)
lib/lp/registry/browser/distroseries.py (+2/-2)
lib/lp/registry/browser/person.py (+1/-1)
lib/lp/registry/browser/product.py (+4/-4)
lib/lp/registry/browser/project.py (+3/-3)
lib/lp/registry/browser/tests/peoplemerge-views.txt (+1/-4)
lib/lp/registry/browser/tests/product-views.txt (+3/-2)
lib/lp/registry/browser/tests/projectgroupset-views.txt (+9/-10)
lib/lp/registry/configure.zcml (+7/-7)
lib/lp/registry/doc/commercialsubscription.txt (+13/-12)
lib/lp/registry/doc/vocabularies.txt (+2/-2)
lib/lp/registry/interfaces/product.py (+4/-4)
lib/lp/registry/stories/distroseries/distroseries-admin.txt (+89/-0)
lib/lp/registry/stories/person/xx-admin-person-review.txt (+17/-19)
lib/lp/registry/stories/product/xx-product-edit.txt (+3/-6)
lib/lp/registry/stories/project/xx-project-edit.txt (+40/-8)
lib/lp/registry/templates/product-index.pt (+1/-1)
lib/lp/registry/templates/project-listing-detailed.pt (+1/-1)
lib/lp/registry/vocabularies.py (+2/-2)
lib/lp/soyuz/doc/distribution.txt (+1/-1)
lib/lp/testing/factory.py (+13/-1)
lib/lp/testing/sampledata.py (+3/-1)
lp:~bac/launchpad/bug-588773-charm
- Edwin Grubbs (community): Approve (code)
- Brad Crittenden: Pending requested
-
Diff: 372 lines (+145/-32)8 files modifiedlib/lp/registry/browser/person.py (+12/-1)
lib/lp/registry/browser/product.py (+16/-3)
lib/lp/registry/browser/project.py (+11/-4)
lib/lp/registry/configure.zcml (+18/-6)
lib/lp/registry/interfaces/projectgroup.py (+14/-9)
lib/lp/registry/stories/person/xx-admin-person-review.txt (+16/-3)
lib/lp/registry/stories/product/xx-product-edit.txt (+33/-5)
lib/lp/registry/stories/project/xx-project-edit.txt (+25/-1)
| Changed in launchpad-registry: | |
| status: | Triaged → In Progress |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Ursula Junque (ursinha) wrote Bug fixed by a commit | #1 |
| tags: | added: qa-needstesting |
Revision history for this message
| Ursula Junque (ursinha) wrote | #2 |
| tags: |
added: qa-bad removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | Fix Committed → Confirmed |
| assignee: | Brad Crittenden (bac) → nobody |
| tags: | removed: qa-bad |
| Changed in launchpad-registry: | |
| milestone: | 10.06 → 10.07 |
| Changed in launchpad-registry: | |
| status: | Confirmed → Triaged |
| tags: |
added: canonical-losa-lp removed: losa |
| Changed in launchpad-registry: | |
| milestone: | 10.08 → 10.09 |
| Changed in launchpad-registry: | |
| assignee: | nobody → Brad Crittenden (bac) |
| status: | Triaged → In Progress |
Revision history for this message
| Brad Crittenden (bac) wrote | #4 |
| tags: |
added: qa-bad removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad-registry: | |
| status: | Fix Committed → In Progress |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| tags: |
added: qa-needstesting removed: qa-bad |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r10966 <http://