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

User Permissions #693

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

User Permissions #693

wants to merge 6 commits into from

Conversation

mohamedelabbas1996
Copy link
Contributor

@mohamedelabbas1996 mohamedelabbas1996 commented Jan 24, 2025

Summary

This pull request implements a fine-grained object-level permissions for the Project model using django-guardian. It ensures that only staff users can create projects, while project owners are granted read, write, and delete permissions. Project members have read and write permissions, and other users are restricted to only read permissions. Additionally, user permissions are included in the API response for every project object. Finally, the project’s object-level permissions can be managed through the Django admin page.

Copy link

netlify bot commented Jan 24, 2025

Deploy Preview for ami-dev canceled.

Name Link
🔨 Latest commit 38addf3
🔍 Latest deploy log https://app.netlify.com/sites/ami-dev/deploys/679724658a17510008d3ad8f

@mohamedelabbas1996
Copy link
Contributor Author

Screenshot 2025-01-27 at 00 51 58 Screenshot 2025-01-27 at 00 54 30

@mohamedelabbas1996 mohamedelabbas1996 marked this pull request as ready for review January 27, 2025 06:00
@mohamedelabbas1996 mohamedelabbas1996 changed the title [Draft] User Permissions User Permissions Jan 27, 2025
@mihow
Copy link
Collaborator

mihow commented Jan 27, 2025

This looks really promising, great work @mohamedelabbas1996. I will review asap!

@mihow mihow mentioned this pull request Jan 27, 2025
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.

2 participants