llaske / sugarizer-server

Sugarizer Server allow deployment of Sugarizer on a local server, for example on a school server.
Apache License 2.0
42 stars 92 forks source link

Delete all user data when the user is deleted #109

Closed NikhilM98 closed 5 years ago

NikhilM98 commented 5 years ago

When a user is deleted, his journals and stats are not removed from the database. I was wondering whether should we remove the user data when the user is deleted. The data can be useful for analytics when we expand sugarizer-server dashboard. But storing someone's data without their consent is also not good.

llaske commented 5 years ago

Good point. I don't think we need to remove stats but journal should certainly be removed.

NikhilM98 commented 5 years ago

@llaske okay. I'll start working on it 🙂

NikhilM98 commented 5 years ago

Fixed in #115