Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the deletedAt Column when a user requests to be deleted #531

Open
3 tasks
eleanorreem opened this issue Jul 18, 2024 · 7 comments
Open
3 tasks

Update the deletedAt Column when a user requests to be deleted #531

eleanorreem opened this issue Jul 18, 2024 · 7 comments
Assignees
Labels
complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days databases Works with databases help wanted Extra attention is needed javascript Pull requests that update Javascript code priority: soon Should be prioritized soon. stale This issue or PR is inactive.
Milestone

Comments

@eleanorreem
Copy link
Contributor

Overview

When a user requests to be deleted they hit the deleteUser method in the user service. This method soft deletes the user and sets them to inactive. We want to record the date in the new user.deletedAt column. The ticket is dependent on #530

Action Items

@eleanorreem eleanorreem added complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days priority: soon Should be prioritized soon. state: blocked Task is blocked. javascript Pull requests that update Javascript code databases Works with databases labels Jul 18, 2024
@eleanorreem eleanorreem changed the title Update the deletedAt Column when a user deletes themselves Update the deletedAt Column when a user requests to be deleted Jul 18, 2024
@kyleecodes kyleecodes added this to the 02. Roadmaps milestone Jul 18, 2024
@kyleecodes kyleecodes added the help wanted Extra attention is needed label Jul 23, 2024
@ckirby19
Copy link
Contributor

Hello, I would like to pick this up once #530 is merged :)

Copy link

Thank you @ckirby19 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 github-actions bot added the stale This issue or PR is inactive. label Sep 23, 2024
@eleanorreem eleanorreem removed the state: blocked Task is blocked. label Oct 1, 2024
@eleanorreem
Copy link
Contributor Author

@ckirby19 ready to go if you are still interested

@ckirby19
Copy link
Contributor

ckirby19 commented Oct 2, 2024

Apologies, I will have a chance to pick this up over the weekend!

@eleanorreem
Copy link
Contributor Author

No rush! Just flagging that this is now unblocked 😄

@ckirby19
Copy link
Contributor

ckirby19 commented Oct 6, 2024

Hello, I was not able to test this properly locally as I wanted to, as I could not get the new database dump to work, with the error
pg_restore: warning: archive is compressed, but this installation does not support compression (this build does not support compression with gzip) -- no data will be available pg_restore: error: cannot restore from compressed archive (this build does not support compression with gzip)
Any suggestions for this?

@kyleecodes
Copy link
Member

Hi @ckirby19, was this error thrown by pgAdmin or a command line tool? Never seen this error before. But this may be an issue with our dump file that needs updated to exclude permission tags. If you follow directions exactly as they are in the Wiki guide (linked in README) it should work regardless.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: moderate Time needed to do this ticket will be moderate e.g. 1-2 days databases Works with databases help wanted Extra attention is needed javascript Pull requests that update Javascript code priority: soon Should be prioritized soon. stale This issue or PR is inactive.
Projects
Development

No branches or pull requests

3 participants