search

incuna / django-user-management

User management model mixins and api views.
BSD 2-Clause "Simplified" License
57 stars 24 forks source link

Add delete on profile view #102

Closed kevinetienne closed 9 years ago

meshy commented 9 years ago

Why?

kevinetienne commented 9 years ago

It looks like we are allowing user deletion in the UserDetail view. Would that make sense to add it to the profile view too? The idea is to be able to allow user to call delete for its profile (for example in djangular-rest-framework)

meshy commented 9 years ago

Aah ok that makes sense. It would be a nice way for users to be able to delete themselves!

I think that it might be wise for us to explicitly set on_delete for all FKs to users in future.

kevinetienne commented 9 years ago

on_delete looks interesting thanks, I wasn't aware about it.

LilyFoote commented 9 years ago

It does look interesting, but I would expect us to want the default CASCADE behaviour in most cases.

kevinetienne commented 9 years ago

updated