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

fix: ajout de contrôles afin d'éviter de renvoyer une exception quand les métadonnées ont un format inattendu #1241

Merged

Conversation

ggrossetie
Copy link
Collaborator

No description provided.

Copy link

netlify bot commented Feb 4, 2025

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit 34dc8b0
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/67a20d0d6bb2680008f9be1a

@thom4parisot
Copy link
Member

Du coup c'est quoi le comportement : ça ignore ces bouts de métadonnées qui ne peuvent être traitées ? Est-ce qu'un passage vers/depuis le formulaire depuis/vers le Yaml fait perdre des données ?

Est-ce qu'on ne ferait pas mieux d'appliquer le schéma sur la conversion Yaml -> JSON et d'avertir là où ça coince ?

@ggrossetie
Copy link
Collaborator Author

Du coup c'est quoi le comportement : ça ignore ces bouts de métadonnées qui ne peuvent être traitées ?

Oui ce n'est pas terrible mais la conversion se fait automatiquement via le resolver de l'API GraphQL.

Est-ce qu'un passage vers/depuis le formulaire depuis/vers le Yaml fait perdre des données ?

Les données ne sont pas perdues mais on ne sait pas les représenter :

image

Est-ce qu'on ne ferait pas mieux d'appliquer le schéma sur la conversion Yaml -> JSON et d'avertir là où ça coince ?

Les données sont en JSON dans la base de données. On peut appliquer/vérifier le JsonShema mais je ne sais pas quelle serait le comportement souhaité ? Est-ce qu'on bloque la prévisualisation ?

@thom4parisot
Copy link
Member

Ok je vois. Ça me va de commencer comme ça et on verra bien s'il faut faire un effort supplémentaire si c'est pas clair/suffisant en terme d'usage 👍🏻

@ggrossetie
Copy link
Collaborator Author

La solution plus long terme c'est de gérer les transformations au niveau du service d'export/prévisualisation. On peut aussi ajouter un contrôle lors de l'enregistrement des métadonnées côté backend Stylo pour garantir que le format correspond à notre JsonSchema.

@thom4parisot thom4parisot merged commit 58928f8 into EcrituresNumeriques:master Feb 4, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants