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

Blog permissions #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

Thominho
Copy link

@Thominho Thominho commented Aug 31, 2020

Editing allows an administrator to assign blog permissions (displays the blog in the menu and gives the user the opportunity to create new posts) through roles. It also prevents the user from editing other users' posts. The user can only edit and create their own posts under their account.

Permission to view and edit only the author's posts. (Permission is granted in roles.)
Permission to view and edit only the author's posts. (Permission is granted in roles.)
@jeroenvanrensen
Copy link
Owner

Hi @Thominho,

Is this PR based upon Issue #1 or not? I can't quite figure it out when looking at your code.

Thank you! Jeroen

@Thominho
Copy link
Author

Hi @Thominho,

Is this PR based upon Issue #1 or not? I can't quite figure it out when looking at your code.

Thank you! Jeroen

Yes. As a result, the user cannot create a post now and save it as another author. (Then the system would redirect it to an error because it would lose the ability to display a "foreign" post.)

@jeroenvanrensen jeroenvanrensen linked an issue Aug 31, 2020 that may be closed by this pull request
@jeroenvanrensen jeroenvanrensen added the enhancement New feature or request label Aug 31, 2020
@jeroenvanrensen
Copy link
Owner

Hi @Thominho,

Okay, now I understand your code.

I also would like if an admin can change the author of the post, so I will change your code a little bit.

Howevery, thank you!

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

Successfully merging this pull request may close these issues.

Delete author selection option
2 participants