Need more permissions on admin template pages for ubuntu-translations-coordinators
Bug #406477 reported by
David Planella
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Adi Roiban | ||
Bug Description
Members of the ubuntu-
https:/
However, once a template has been disabled, the +admin page is no longer accessible to them, which stops them from e.g. enabling the template again in case of a mistake.
Related branches
lp://qastaging/~adiroiban/launchpad/bug-406477
- Данило Шеган (community): Approve
-
Diff: 708 lines (+380/-101)11 files modifiedlib/canonical/launchpad/icing/style-3-0.css (+13/-0)
lib/canonical/launchpad/security.py (+74/-33)
lib/lp/testing/factory.py (+4/-1)
lib/lp/translations/browser/configure.zcml (+1/-1)
lib/lp/translations/browser/potemplate.py (+2/-2)
lib/lp/translations/configure.zcml (+1/-1)
lib/lp/translations/stories/distroseries/xx-distroseries-templates.txt (+72/-15)
lib/lp/translations/stories/standalone/xx-potemplate-admin.txt (+42/-2)
lib/lp/translations/stories/standalone/xx-potemplate-index.txt (+70/-1)
lib/lp/translations/templates/object-templates.pt (+100/-44)
lib/lp/translations/templates/potemplate-index.pt (+1/-1)
| Changed in rosetta: | |
| status: | New → Triaged |
| importance: | Undecided → High |
Revision history for this message
| Adi Roiban (adiroiban) wrote | #1 |
| Changed in rosetta: | |
| milestone: | none → 3.1.12 |
| Changed in rosetta: | |
| assignee: | nobody → Adi Roiban (adiroiban) |
| status: | Triaged → Fix Committed |
| tags: | added: qa-needstesting |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is an first attempt in solving this bug.
The translations testsuite was OK, but I will need to add new tests. Since the templates pagetest will overlap with a branch that is not yet pushed to devel, I will add the merge proposal after that branch is merged.