move the CustomerUser delete logic from the view to the model - Githubissues

astrosat / django-astrosat-users

Common backend library for Astrosat projects' user management

GNU General Public License v3.0

2 stars 0 forks source link

move the CustomerUser delete logic from the view to the model #80

Closed allynt closed 4 years ago

allynt commented 4 years ago

Checklist

[ ] Is this issue linked to an Epic issue?
[ ] Have you linked each commit to their issue e.g. First line should contain text like IssueID #123 - Update dependencies for front-end:. The important bit is the start IssueID #123, this will create a reference in the issue, to the commit.
[ ] Have you set Reviewers, labels and Milestone?

Is your feature request related to a problem? Please describe

Move the CustomerUser delete logic from the view to the model.

Describe the solution you'd like

Just like the invite method, there should be an uninvite method.

Describe alternatives you've considered

If I ever get viewsets working (#79), then this could be another @action on that viewset.

allynt commented 4 years ago

Also bottom-out under which circumstances the associated user should be deleted.

allynt commented 4 years ago

Done in #94