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

Guardian setup #750

Merged
merged 5 commits into from
Jul 22, 2024
Merged

Guardian setup #750

merged 5 commits into from
Jul 22, 2024

Conversation

dpgraham4401
Copy link
Member

Description

adds Django Guardian as a dependency and set up object related permissions with direct foreign keys (instead of ContentType) for our Site and Org model.

Issue ticket number and link

closes #743

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@github-actions github-actions bot added database changes that require modification to the database schema dependencies project dependency problems or modifications server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included labels Jul 22, 2024
@dpgraham4401 dpgraham4401 merged commit 00124bf into USEPA:main Jul 22, 2024
7 checks passed
@dpgraham4401 dpgraham4401 deleted the guardian_setup branch July 22, 2024 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database changes that require modification to the database schema dependencies project dependency problems or modifications server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UserRole model
1 participant