MTES-MCT / dialog

Intégration de la réglementation de circulation dans les solutions numériques
https://dialog.beta.gouv.fr
GNU Affero General Public License v3.0
8 stars 0 forks source link

Add regulation order issues admin #818

Closed mmarchois closed 2 weeks ago

mmarchois commented 2 weeks ago

image

Cette PR est une première implémentation de l'espace d'administration des alertes des arrêtés. Dans une première version, elle permet de lister et de supprimer les alertes remontées. En fonction des retours de @johanricher et @MathieuFV qui l'utiliseront, des évolutions pourront être apportées.

codecov-commenter commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.11%. Comparing base (c9a7b63) to head (72feb70).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #818 +/- ## ========================================= Coverage 99.11% 99.11% Complexity 1174 1174 ========================================= Files 219 219 Lines 4947 4947 ========================================= Hits 4903 4903 Misses 44 44 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

florimondmanca commented 2 weeks ago

Proposition : plutôt passer par Sentry pour lever des erreurs de qualité des données, avec un tag approprié genre type:dataquality

On peut mettre l'identifiant et tout autre élément de contexte en tag, regulation_identifier:2024T230945, etc

Et bénéficier de tous les outils d'analyse de Sentry avec un travail de développement réduit à la connexion entre DiaLog et Sentry (préparation et envoi des événements à logger)

mmarchois commented 2 weeks ago

Suite à une discussion avec @florimondmanca , nous avons choisi d'utiliser, au moins dans un premier temps, l'outil de monitoring Sentry.