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

store masked permissions on user model #77

Open
unenglishable opened this issue Oct 27, 2023 · 0 comments
Open

store masked permissions on user model #77

unenglishable opened this issue Oct 27, 2023 · 0 comments

Comments

@unenglishable
Copy link
Member

get_masked_permissions is called in format_user_reply as well as in helpers/acl
(any time permissions need to be calculated)

this is expensive and unnecessary if we consolidate by storing the calculated value on the user model itself in the database
this will lower the time complexity to read and remove any duplicate calls

create: store on model in db
update: when user/role is updated

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

No branches or pull requests

1 participant