search

etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports
https://transport.data.gouv.fr
192 stars 30 forks source link

NotificationSubscriptionProducerJob : supprime puis recrée les abonnements #3912

Closed AntoineAugusti closed 5 months ago

AntoineAugusti commented 5 months ago

En lien avec #3896

Implémente cette partie.

Quand quelqu’un passe de réutilisateur à producteur, supprimer ses notifs au lieu de les modifier et recréer toutes les notifs de l’autre rôle

Ce refactor permet d'utiliser DB.NotificationSubscription.changeset et gère la différence des motifs entre réutilisateur et producteur (avant on changeait juste le rôle).

La suppression des favoris (ancien réutilisateur qui suit un JDD dont il est producteur désormais) est gérée dans un autre job https://github.com/etalab/transport-site/blob/679085fcc7b73b805f5f33adbdd4ad32c5153e51/apps/transport/lib/jobs/import_dataset_followers_job.ex#L22

vdegove commented 5 months ago

C’était vraiment très direct et clair à lire :)