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

Ajout d'une commande pour supprimer les utilisateurs inactifs #1001

Merged
merged 3 commits into from
Jul 12, 2021

Conversation

mininao
Copy link
Member

@mininao mininao commented Jul 9, 2021

Résumé

Supprime les utilisateurs inscrits depuis plus de X semaines, ayant refusé/ignoré plus de X notifications, et leur envoie un mail.

Détails

  • C'est pas ultra propre, mais je pense juste la faire tourner quelques fois à la main, d'où les puts qui trainent.

@carsso
Copy link
Member

carsso commented Jul 9, 2021

Ton test ne passe pas
rspec ./spec/jobs/send_inactive_user_emails_job_spec.rb:50 # SendInactiveUserEmailsJob.inactive_user_ids when a user have at least a refused match excludes users having refused matches, they are considered active users

lib/tasks/emails.rake Outdated Show resolved Hide resolved
@mininao mininao requested a review from carsso July 12, 2021 11:33
@carsso carsso enabled auto-merge (squash) July 12, 2021 12:09
@carsso carsso merged commit 3b4f190 into main Jul 12, 2021
@carsso carsso deleted the maxence/inactive-user-removal branch July 12, 2021 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants