-
Notifications
You must be signed in to change notification settings - Fork 17
BC-8635 - fix task permission #5417
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
Conversation
when the task had no course or lesson it was possible for another teacher to gain permission to the task
…schul-cloud/schulcloud-server into BC-8635-fix-task-permission
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't find a reason for hasParentPermission to default return true.
Best guess is that it is a copy paste fragment.
Good catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can remove the comment in line 45:
// TODO why parent permission has OR cond?
when the task had no course or lesson it was possible for another teacher to gain permission to the task
when the task had no course or lesson it was possible for another teacher to gain permission to the task
Description
when the task had no course or lesson
it was possible for another teacher
to gain permission to the task
Links to Tickets or other pull requests
BC-8635
Approval for review
generate-client:server
was executed in vue frontend and changes were tested and put in a PR with the same branch name.