On 9/21/18 5:32 PM, Adel Atallah wrote:
On Fri, Sep 21, 2018 at 4:56 PM Simon Urli <[email protected]> wrote:
On 9/21/18 4:53 PM, Adel Atallah wrote:
+1 for the warning, but I would not forbid simple users from renaming
or moving pages but instead just hide the action (from the page menu).
OK I should have written it: by "forbid" I meant:
1. Hide the action from the menu
2. Return an error message if the user try to access the
renaming/moving page (using forged URL)
So you suggest we shouldn't do 2?
That's what I suggest yes, being an advanced user is just a matter of
changing a setting in your profile AFAIK so I don't see why a simple
user shouldn't be able to access the same pages as the advanced one
(at least with the URL).
LGTM. So the users would see the same warning as the advanced users then.
Other feedback?
On Fri, Sep 21, 2018 at 4:44 PM Simon Urli <[email protected]> wrote:
Hi all,
users might currently break their AWM application by renaming/moving
pages containing XClass definition.
We need a proper refactoring operation to be able to properly do such
move/rename. But this feature might take a while to be completely
available.
In the meantime I propose that we prevent users from renaming/moving
pages containing XClass.
What I propose is the following:
- Forbid completely *simple users* to rename/move pages containing XClass
- Display a warning to *advanced users* when they perform such
operation: the same kind of warning we already have when performing edit
on XWiki pages
WDYT?
Simon
--
Simon Urli
Software Engineer at XWiki SAS
[email protected]
More about us at http://www.xwiki.com
--
Simon Urli
Software Engineer at XWiki SAS
[email protected]
More about us at http://www.xwiki.com
Thanks,
Adel
--
Simon Urli
Software Engineer at XWiki SAS
[email protected]
More about us at http://www.xwiki.com
