Solution de suivi des problèmes qualité des arrêtés

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

Solution de suivi des problèmes qualité des arrêtés #813

Open mmarchois opened 3 weeks ago

mmarchois commented 3 weeks ago

User story

Suite aux points concernant la sécurité (voir le compte rendu sur le pad) et les dates / horaires des mesures non intégrées (#812), il a été convenu de créer une interface utilisateur accessible depuis l'administration pour lister les erreurs liées à la création/intégration d'un arrêté.

À la manière d'un Sentry, l'objectif est de pouvoir historiser et identifier rapidement les erreurs lors de la création d'un arrêté afin de nous assurer de la qualité des données exportées par DiaLog.

Implémentation

[ :warning: Pas à jour]

[ ] Création d'une nouvelle table RegulationOrderIssue contenant :
- uuid
- level (warning / error)
- identifier
- organization
- context
- source
- geometry
- created_at
[ ] Affichage des issues dans l'administration
[ ] Suppression d'une issue depuis l'administration
[ ] Tests

florimondmanca commented 3 weeks ago

Ref https://github.com/MTES-MCT/dialog/issues/812#issuecomment-2149851098

mmarchois commented 2 weeks ago

@johanricher est-ce que dans un premier temps ce genre de vue te convient ?

johanricher commented 2 weeks ago

Oui très clair comme UI. Quelle est l'origine de l'info "erreur" ou "warning" ? Est-ce que l'approche serait d'associer chaque alerte avec un rapport d'erreur / warning qui serait dans un log, par exemple avec un lien vers les jobs d'import dans Github Actions ?

mmarchois commented 2 weeks ago

Quelle est l'origine de l'info "erreur" ou "warning"

Il s'agit d'un niveau de criticité. Pour le moment, l'idée serait de dire que les arrêtés comportant une erreur ne seront pas importés (par exemple, impossibilité de parser une date), ceux ayant un warning seront quand même importé (par exemple, hors zone géographique) mais remonté dans l'admin pour pouvoir faire une validation à postériori.

florimondmanca commented 2 weeks ago

Vu ce qu'on s'est dit ce matin concernant l'utilisation de Sentry comme solution technique,

Je propose de renommer / retravailler ce ticket pour parler du besoin utilisateur côté contrôle de la qualité des données, tq @johanricher en a parlé

Avoir accès à des "rapports d'erreur" concernant la qualité des données : problèmes constatés, criticité, date, autres informations pertinentes (identifiant arrêté, etc)
Pouvoir en faire une analyse basique