Closed dannygoldstein closed 3 years ago
Hello @dannygoldstein! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
app/models.py
:Line 639:80: E501 line too long (80 > 79 characters) Line 641:80: E501 line too long (80 > 79 characters) Line 687:80: E501 line too long (82 > 79 characters)
This PR adds a custom
UserAccessControl
that allows developers to specify complex access permissions such as those required for group/stream access in SkyPortal as SQLalchemyQuery
objects.It also fixes a bug where I was only automatically checking the permissions of objects that were being deleted instead of deleted, read, or updated.