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

Access Copy Page function for a role that only has view only for 'All' but can create their own. #1199

Closed
mark-james opened this Issue Dec 28, 2018 · 2 comments

Comments

2 participants
@mark-james
Copy link
Contributor

mark-james commented Dec 28, 2018

Describe the bug
If a user only has view access on a book page, the user should still be able to copy that page to a book that they do have create/update access for.

Steps To Reproduce
Steps to reproduce the behavior:

  1. Create a role with Create/Update for own books/pages etc
  2. Add View access to other books to the same role
  3. Login as user with that role
  4. Go to view only book. The 'More' menu on the right hand side is not available.

Expected behavior
For a view only page a user should be able to 'copy' that page to a book provided they have create/update access on at least one book.

Screenshots
Copy Functionality
image

Additional context
Use Case: I have a bunch of pages that are templates. I don't want users to be able to edit them. But I do want them to be able to take a copy and then edit that copy.

@ssddanbrown

This comment has been minimized.

Copy link
Member

ssddanbrown commented Dec 28, 2018

@mark-james Thanks for the suggestion. I can see the use-case for this and can see how this would be useful. I agree this is something that would be ideal to have.

Since you could also copy to chapters the copy option would need to show for users with create access to any chapter or book.

Is a bit different to other permission checks so will have to ensure it's done with performance in mind but should be possible.

@ssddanbrown

This comment has been minimized.

Copy link
Member

ssddanbrown commented Mar 9, 2019

Now implemented as of merge in #1202. To be in the next patch release.

@ssddanbrown ssddanbrown closed this Mar 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.