Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve default permissions of Drafts page #2246

Merged
merged 1 commit into from Apr 16, 2015
Merged

Improve default permissions of Drafts page #2246

merged 1 commit into from Apr 16, 2015

Conversation

hissy
Copy link
Contributor

@hissy hissy commented Apr 14, 2015

Draft pages inherits their permissions from their parent page (usually it is Home page). When we want to create a limited user account that will be allowed to create some types of page, we have to change the permissions of Draft (/!drafts) page. If not, the limited user will get weird "Not Found" page when creating a new draft page, because the user can't access to editing permissions of "Home" page. It is not intuitive. This change helps people who wants to setup Advanced Permission mode.

  1. /!drafts page should have own permissions, not inherit from Home page
  2. Assign "Page Owner" to some permissions that are required to access to Composer interface
  3. A new draft page inherits its permissions from /!drafts page.

@aembler
Copy link
Member

aembler commented Apr 16, 2015

Nice catch

aembler added a commit that referenced this pull request Apr 16, 2015
Improve default permissions of Drafts page
@aembler aembler merged commit 35b6075 into concretecms:develop Apr 16, 2015
@hissy hissy deleted the feature/permission-drafts branch April 16, 2015 03:50
@hissy
Copy link
Contributor Author

hissy commented Apr 16, 2015

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants