-
Notifications
You must be signed in to change notification settings - Fork 256
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
update permissions for write access #946
Conversation
Codecov Report
@@ Coverage Diff @@
## master #946 +/- ##
==========================================
- Coverage 54.24% 52.72% -1.53%
==========================================
Files 181 181
Lines 8850 8924 +74
==========================================
- Hits 4801 4705 -96
- Misses 4049 4219 +170
Continue to review full report at Codecov.
|
return ( | ||
self.has_object_read_permission(request) | ||
and request.user.user_type == User.TYPE_VALUE_MAP["Doctor"] | ||
or request.user.user_type >= User.TYPE_VALUE_MAP["DistrictAdmin"] |
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.
This overrides the check for dist admins right? False and False or True is True
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.
Yup, I messed up the logic 😅
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.
Updated it
Why the force pushes ? |
I thought that it would have been better to have a single commit for smaller changes. |
Kudos, SonarCloud Quality Gate passed!
|
All commits will end up being squashed to one, so no worries! |
fixes: #938