microsoft / azurechat

🤖 💼 Azure Chat Solution Accelerator powered by Azure Open AI Service
MIT License
1.15k stars 958 forks source link

DeletedAt to replace IsDeleted? #266

Open jimmystridh opened 7 months ago

jimmystridh commented 7 months ago

Hi,

To be able to have good GDPR compliant setup we would like to wipe our users' deleted threads and messages after say 14 days (but possibly keep them around soft-deleted for compliance, security, debugging etc). Currently there is no way of doing that since there is no way to know when a certain thread or message was deleted.

If we were to submit a pull request replacing isDeleted with deletedAt throughout the code, deletedAt having a value if the entity is deleted, or null otherwise, would this be an agreeable solution?

Thank you for your input