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

Add metadata permissions #107

Open
wants to merge 15 commits into
base: integration
Choose a base branch
from
Open

Conversation

dxenes1
Copy link
Contributor

@dxenes1 dxenes1 commented Nov 8, 2022

Permission modifications to separate resource and metadata permissions.

Use cases:

  1. Allow external collaborators to write metadata but not modify public spatial data.
  2. Allow groups to manage metadata to update the website automatically.
  3. Allow secondary analysis groups to access and write metadata for projects they are working with

Permission Groups

read resource add resource update resource delete resource read metadata add metadata update metadata delete metadata
read x x
read+addmeta x x x
read+fullmeta x x x x x
write x x x x x x x
admin x x x x x x x
admin+delete x x x x x x x x

django/bosscore/models.py Outdated Show resolved Hide resolved
django/bosscore/permissions.py Outdated Show resolved Hide resolved
django/mgmt/utils.py Outdated Show resolved Hide resolved
@dxenes1
Copy link
Contributor Author

dxenes1 commented Dec 22, 2022

I think all requested changes should be addressed in the latest commit, but please let me know if I missed anything!

@dxenes1 dxenes1 self-assigned this Dec 22, 2022
movestill
movestill previously approved these changes Jan 3, 2023
Copy link
Member

@movestill movestill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see lots of old code that could use refactoring, but I think your changes for metadata look good!

@dxenes1 dxenes1 changed the base branch from dev-stack-fix to integration January 31, 2023 18:45
@dxenes1 dxenes1 dismissed movestill’s stale review January 31, 2023 18:45

The base branch was changed.

@dxenes1 dxenes1 changed the base branch from integration to dev-stack-fix January 31, 2023 18:46
@dxenes1 dxenes1 changed the base branch from dev-stack-fix to integration February 16, 2023 20:39
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