Closed
Bug 675842
Opened 13 years ago
Closed 11 years ago
Permission for moving pages
Categories
(developer.mozilla.org Graveyard :: User management, task, P3)
developer.mozilla.org Graveyard
User management
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: openjck, Unassigned)
References
Details
(Whiteboard: [user-interview][type:feature][triaged])
Please see the following comment for the motivation behind this request: https://bugzilla.mozilla.org/show_bug.cgi?id=675795#c0
Reporter | ||
Updated•13 years ago
|
Priority: P4 → P3
Reporter | ||
Comment 1•13 years ago
|
||
This permission should not be granted to ordinary users.
Depends on: 677814
Summary: Do not allow ordinary users to move pages → Permission for moving pages
Reporter | ||
Updated•13 years ago
|
Whiteboard: u=administrator c=siteadmin p= → [user-interview] u=administrator c=siteadmin p=
Updated•13 years ago
|
Comment 2•13 years ago
|
||
This is currently open to all, in Kuma. To gate it with access control, I think we'd wrap the ability to change title and slug with a permission. Slug actually changes the URL, but we might as well cover title too. One question: Should a user be able to move/rename a page he or she created?
Comment 3•13 years ago
|
||
I think yes. I don't know if Kuma still has a "user sandbox", but a user should be able to create a page in a scratch area, and move it into its "real" location when it's "done". Or they may realize that they created a page in the "wrong" location; letting them move it without bugging anybody would be good. In that case, the key thing is making the move dialog more self-explanatory than in MindTouch. MT's move dialog confuses people and causes them to do the wrong thing.
Assignee | ||
Updated•12 years ago
|
Version: Kuma → unspecified
Assignee | ||
Updated•12 years ago
|
Component: Website → Landing pages
Reporter | ||
Updated•11 years ago
|
Component: Landing pages → User management
Whiteboard: [user-interview] u=administrator c=siteadmin p= → [user-interview]
Reporter | ||
Comment 4•11 years ago
|
||
Les, do you think we should keep this open? IIRC we manage this "permission" with Waffle currently, but I know that may not be the best solution.
Flags: needinfo?(lorchard)
Reporter | ||
Updated•11 years ago
|
Severity: normal → enhancement
Whiteboard: [user-interview] → [user-interview][type:feature][triaged]
Comment 5•11 years ago
|
||
(In reply to John Karahalis [:openjck] from comment #4) > Les, do you think we should keep this open? IIRC we manage this "permission" > with Waffle currently, but I know that may not be the best solution. Waffle should only ever be thought of as a temporary gate to a feature in testing. We should never consider it permission control.
Flags: needinfo?(lorchard)
Comment 6•11 years ago
|
||
We need to get this tied to the "Can move a tree of pages" permission now, so we can start letting trusted localizers move pages to keep up with the massive reorg the English content is undergoing. Can we get this done in the next few days, please?
Comment 7•11 years ago
|
||
This would be good, even if we didn't foresaw how much this would be needed with the reorg; ideally we would like to have this for next Saturday (Paris l10n Sprint Week-end) Feedbacking Maris.
Flags: needinfo?(mars)
Comment 8•11 years ago
|
||
James, Les, what do you think about comment 6? Would it be possible to tie the permissions together in time for the docs sprint on April 26th?
Flags: needinfo?(mars)
Flags: needinfo?(lorchard)
Flags: needinfo?(jbennett)
Comment 9•11 years ago
|
||
(In reply to MΔris Fogels [:mars] from comment #8) > James, Les, what do you think about comment 6? Would it be possible to tie > the permissions together in time for the docs sprint on April 26th? I'm not really familiar with page move, but what we could probably do by then is: 1) Wrap the feature in a Django permission 2) Create a new user group, grant the permission to the group 3) Add users as needed to the group via Django admin That would require whomever wants to share the permission to get comfortable navigating Django admin views & adding users to a group. Anything more user friendly is on a much farther horizon.
Flags: needinfo?(lorchard)
Comment 10•11 years ago
|
||
I can get this together in the next couple days.
Flags: needinfo?(jbennett)
Updated•11 years ago
|
Assignee: nobody → jbennett
Updated•11 years ago
|
Assignee: jbennett → nobody
Comment 11•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/kuma https://github.com/mozilla/kuma/commit/70b658ba3b9d6567309f91ae51bb01305276ccd6 Bug 675842: Remove waffle flag for page move. This removes the waffle flag check that controls access to the page move view; going forward, access will be controlled by the permissions infrastructure. https://github.com/mozilla/kuma/commit/eb926aefdab2fdaa78c2ad7998c94f66ccb0459e Merge pull request #2322 from ubernostrum/page-move-no-waffle Bug 675842: Remove waffle flag for page move.
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•4 years ago
|
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•