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

Modified User Model actions #97

Closed
wants to merge 11 commits into from
Closed

Conversation

siddnikh
Copy link
Contributor

@siddnikh siddnikh commented May 7, 2023

  • Modified User Model to have a user_type with the following specifications:
    Admin - has read and write permissions for user (3)
    Supervisor - has read only permissions for user (2)
    User - read and write permissions for only their user object (1)

The CRUD operations were already implemented for the User Model, but added permission to check so only django superadmins and admin users could access them. A user can also access the CRUD endpoint but only for their own user object.

@siddnikh siddnikh requested a review from skks1212 May 7, 2023 08:26
@siddnikh siddnikh linked an issue May 7, 2023 that may be closed by this pull request
ayushma/models/users.py Outdated Show resolved Hide resolved
ayushma/models/users.py Outdated Show resolved Hide resolved
ayushma/models/users.py Outdated Show resolved Hide resolved
@siddnikh siddnikh requested a review from skks1212 May 21, 2023 09:29
@skks1212 skks1212 closed this May 30, 2023
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.

Add CRUD operations for User
2 participants