Create Deleted At Column in the database and back fill user data

chaynHQ / bloom-backend

Code for the backend / API of the Bloom service

https://bloom.chayn.co/

MIT License

18 stars 31 forks source link

Create Deleted At Column in the database and back fill user data #530

Open eleanorreem opened 3 months ago

eleanorreem commented 3 months ago

Overview

When a user requests to be deleted, we soft delete them and hash their personal details. If we want to find a list of dates users were deleted, we can guess when the user was deleted by seeing if the user is set as user.active=false and take the date in the updatedAt column. However, we would prefer for this data to have its own column. We want to also add a nullable column to the user called "deletedAt" so we can have the date of when the user was deleted for monitoring purposes. We also want to backfill the data

Action Items

[ ] Create the deletedAt column on the user table
[ ] Write a migration which sets the uesr.deletedAt column to the date of user.updatedAt where the user.active is set to false
[ ] Create a PR and tag @eleanorreem and @annarhughes as reviewers

srodif commented 3 months ago

Hello! Could you assign this issue to me? Thanks in advance!

github-actions[bot] commented 2 months ago

Thank you @eleanorreem you have been assigned this issue! Please follow the directions in our Contributing Guide. We look forward to reviewing your pull request shortly ✨

Support Chayn's mission? ⭐ Please star this repo to help us find more contributors like you! Learn more about Chayn here and explore our projects. 🌸

github-actions[bot] commented 2 months ago

Thank you @eleanorreem you have been assigned this issue! Please follow the directions in our Contributing Guide. We look forward to reviewing your pull request shortly ✨

Support Chayn's mission? ⭐ Please star this repo to help us find more contributors like you! Learn more about Chayn here and explore our projects. 🌸

eleanorreem commented 2 months ago

@srodif accidentally assigned myself. Ignore the bot - you are assigned!

srodif commented 2 months ago

Hello @eleanorreem! I have created a pull request for this issue. Should I add something else?

srodif commented 1 month ago

Just to note that I continue to work on this, for the second action item.

eleanorreem commented 4 weeks ago

@srodif I didn't see your comment that you were still working on it - apologies! Can you make a separate PR for the migration work? Thank you!