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 SATS support #764

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Add SATS support #764

wants to merge 7 commits into from

Conversation

Muirrum
Copy link
Member

@Muirrum Muirrum commented Mar 28, 2023

Code contributed as part of the SATS ISP.

Closes #729

Create models and migrations for SATS as well as modifications to the
user class
@Muirrum
Copy link
Member Author

Muirrum commented Mar 28, 2023

Closes #729

@coveralls
Copy link

coveralls commented Mar 28, 2023

Coverage Status

Coverage: 92.973% (+0.02%) from 92.958% when pulling 7e83c50 on sats into ccde804 on master.

Muirrum added 6 commits April 11, 2023 13:32
Include trailing slash
Only allow actions to be taken by users who have the permissions on the
underlying models. This is a difference from other ModelViewsets, which
allowed any action by any authorized user.
Also allow asset_archived to be unset, will default to False
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants