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.

Sign up for GitHub

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

Jump to bottom

Schedule deletion of users #1872

Merged
merged 5 commits into from
Sep 10, 2023
Merged

Schedule deletion of users #1872

merged 5 commits into from
Sep 10, 2023

Conversation

acasajus
Copy link
Collaborator

@acasajus acasajus commented Sep 7, 2023

users to be scheduled to be deleted cannot:

  • receive or send emails
  • login

For now the cron is disabled in case this gets deployed to prod before extensive tests

nguyenkims
nguyenkims reviewed Sep 8, 2023
View reviewed changes
Copy link
Contributor

@nguyenkims nguyenkims left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just 1 comment.

cron.py Outdated Show resolved Hide resolved
@nguyenkims nguyenkims merged commit 373c30e into master Sep 10, 2023
3 checks passed
@nguyenkims nguyenkims deleted the user_schedule_deletion branch September 10, 2023 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nguyenkims nguyenkims nguyenkims left review comments

@cquintana92 cquintana92 Awaiting requested review from cquintana92 cquintana92 is a code owner

Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@acasajus @nguyenkims