search

etalab / transport-site

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

NotificationSubscription : migre platform-wide vers reuser #3906

Closed AntoineAugusti closed 2 months ago

AntoineAugusti commented 2 months ago

En lien avec #3896

Modifie le role à reuser pour les abonnements existants aux motifs platform-wide qui avaient précédemment un role à producer, l'abonnement ayant été créé depuis le backoffice. https://github.com/etalab/transport-site/pull/3899 ajoute une validation de la cohérence (reason / role) à terme.

Seul le combo reason=daily_new_comments et role=producer n'est pas migré. Il doit rester ainsi, il y a 2 abonnements existants (le PAN et l'ART). Le code gère bien le cas producer et reuser

https://github.com/etalab/transport-site/blob/48d10ae88b4df656ef4cfefa4edbbc8c800d2987/apps/transport/lib/transport/comments_checker.ex#L62-L71

AntoineAugusti commented 2 months ago

Merci @ptitfred pour la review !

Je mergerai en début de semaine prochaine.