-
Notifications
You must be signed in to change notification settings - Fork 132
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
Propagate project permissions to all KPI Assets #2336
Conversation
1a87b55
to
94d3a55
Compare
acdf99f
to
3fa3ac7
Compare
- Add `retrieve_asset_permissions` & `assign_change_asset_permission` utility functions - Add mechanism to remove permissions from users who no longer have a role tied to the project
…rmission functions
6665203
to
80a8a2c
Compare
80a8a2c
to
c987649
Compare
e009328
to
c987649
Compare
c6ba34b
to
e246be0
Compare
# Propagate permissions for XForms that were published by | ||
# Formbuilder | ||
for form_id in form_ids: | ||
asset: XForm = XForm.objects.get(id=form_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add a try catch block here so that if the xform doesnt exist in replica it can be fetched from master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that might not be something we'd need to handle since this line should only return XForm objects that are available in the moment
Changes / Features implemented
Steps taken to verify this change does what is intended
Side effects of implementing this change
TODO
Before submitting this PR for review, please make sure you have:
Closes #1060