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

Entity lists #482

Draft
wants to merge 5 commits into
base: develop
Choose a base branch
from
Draft

Entity lists #482

wants to merge 5 commits into from

Conversation

martastain
Copy link
Member

No description provided.

@martastain martastain linked an issue Jan 8, 2025 that may be closed by this pull request
@martastain
Copy link
Member Author

List ACL idea:
Every list will have "access" field, containing {user: accessLevel} dictionary. Access level is a simple integer

class ListAccessLevel(IntEnum):
    READ = 10 # Can view the list and its items
    UPDATE = 20 # Can update attributes of the list items
    CONSTRUCT = 30 # Can add/remove items from the list or materialize new items
    ADMIN = 40 # Can update/delete the list itself and add new users to the list

So all checks could be performed like:

if entity_list.access.get(user.name, 0) < 30: 
    raise ForbiddenException("You cannot update this list's items")

@martastain martastain self-assigned this Feb 7, 2025
@martastain martastain added epic A large feature set consisting of multiple issues and PRs (issues only) type: feature Adding something new and exciting to the product labels Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic A large feature set consisting of multiple issues and PRs (issues only) type: feature Adding something new and exciting to the product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Entity lists
1 participant