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

Notes non voulues dans le descriptif #510

Closed
DaFrenchFrog opened this issue Oct 19, 2022 · 2 comments
Closed

Notes non voulues dans le descriptif #510

DaFrenchFrog opened this issue Oct 19, 2022 · 2 comments
Labels
bug Something isn't working

Comments

@DaFrenchFrog
Copy link
Collaborator

DaFrenchFrog commented Oct 19, 2022

Description du bug

Des informations (semble-t-il liées à l'import) sont présentes dans la zone de description.
Exemple ici
image

@DaFrenchFrog DaFrenchFrog added the bug Something isn't working label Oct 19, 2022
@florimondmanca
Copy link
Collaborator

Hello @DaFrenchFrog, cette série de signalements de bugs provient des utilisateurs MC j'imagine ?

Pour info ces "notes d'import" ont été ajoutés à dessein car sinon on perdait l'information originale en raison des enums trop restrictifs, voir #456 (comment)

Si le MC est OK pour perdre cette info, une manip en DB peut permettre de retirer ces "notes", et il faudra que je modifie #504 pour ne plus les ajouter pour les futurs imports.

Sinon, je dirais qu'il faut les inviter à rechercher "Notes d'import automatique" via la recherche, et à modifier les jeux de données en mettant la valeur pertinente... puis retirer les notes de la description.

Ça pose la question de l'UX d'édition "à grande échelle", on dirait. À quel point est-il facile de trouver un jeu de données ciblé et de le modifier "rapidement" ?

@johanricher
Copy link
Member

johanricher commented Oct 19, 2022

Oui ça avait été validé avant l'import. Cela a révélé les problèmes évoqués à cette occasionssur les champs concernés, notamment la valeur "quinquennale" pas prévue par l'enum du champ fréquence. Comme on ne pouvait pas résoudre les problèmes rapidement (l'import des données était attendu), la solution proposé par Florimond a été adopté (je n'en vois pas de meilleure).

La seule piste pour résoudre le problème à court/moyen terme c'est d'itérer sur les champs concernés (supprimer les enum, améliorer l'implé, etc.) et, si besoin, refaire un import.

Il faut que tout le monde ait conscience que s'il y a un nouvel import, les modifications qui auront lieu entretemps sur le catalogue du MC sur catalogue.data.gouv.fr seront alors écrasées. J'ai conscience que le fait d'attendre pour un nouvel import est problématique pour les utilisateurs. Il faut à mon avis peser la nécessité de faire un nouvel import par rapport aux données qui ne sont pas conformes au schéma actuellement implémenté ("quinquennal" est un exemple mais il y en a plein d'autres !). Pas d'autre solution que de voir au cas par cas je pense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants