grahamjenson / ger

Good Enough Recommendation (GER) Engine
376 stars 47 forks source link

Delete Person #52

Open hashgupta opened 7 years ago

hashgupta commented 7 years ago

Is there a way to delete people from the database. This could be helpful in a web app when a user deletes their account.

grahamjenson commented 7 years ago

The only reference to users in the system are the events they are in and there is a method delete_events(namespace, person, action, thing) where you could pass delete_events('movies', 'bob') to delete all of bobs movie events.

If you do delete a users events then recommendations across the system may become worse. There are laws around the world where you must delete any information you have on a user. You might be able to selectively anonymize all user activity by using a random uuid instead of their name, and if a user deletes their account then delete the uuid, this way it is difficult to associate any event with a particular deleted user.

grahamjenson commented 7 years ago

Looking at the code delete_events may be broken. I will attempt to fix that soon.