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

Formatter les métadonnées lors des exports #311

Closed
5 tasks done
thom4parisot opened this issue Feb 12, 2021 · 4 comments · Fixed by #312
Closed
5 tasks done

Formatter les métadonnées lors des exports #311

thom4parisot opened this issue Feb 12, 2021 · 4 comments · Fixed by #312

Comments

@thom4parisot
Copy link
Member

thom4parisot commented Feb 12, 2021

Pourquoi : dans la continuité de #270, et indépendamment de #280.

La stratégie est de garder les schémas "purs" (#147, et #192) : on stocke la donnée une fois, on la duplique en sortie.

  • transformer date en year (YYYY), month (MM) et day (DD)
  • transformer date (YYYY/MM/DD) en date (YYYY-MM-DD, format ISO)
  • transformer keywords (formattage, liste concaténée)
  • le formattage des mots-clés doit résoudre Trouver un meilleur moyen de séparer les mots-clés #295
  • générer le text à partir de text_f en supprimant la syntax Markdown (on ne garde que le texte)
keywords:
- lang: fr
  list:
    - **Tomme de Savoie**
    - Comté

en …

{
	"keywords": [
		{
			  "lang": "fr",
			  "list": "Tomme de Savoie, Comté",
			  "list_f": "**Tomme de Savoie**, Comté"
		}
	]
}

Pour tester :

@thom4parisot
Copy link
Member Author

@antoinentl je reprends la conversation ici — c'est difficile pour moi de suivre une conversation sur une PR déjà fusionnée (ça transforme une proposition de code terminée en "issue" qui collecte des feedbacks), et de planifier sa résolution.

@antoinentl
Copy link
Member

Pour tester l'article la solution choisie avec @Mogztter ce serait de :

  1. réintégrer cet article test avec ses métadonnées
  2. tester l'export ZIP avec une comparaison du fichier YAML exporté et l'export HTML pour vérifier que les métadonnées sont toutes là
  3. déployer en dev
  4. tester à nouveau
  5. déployer en production si c'est ok

@antoinentl
Copy link
Member

Une question supplémentaire que nous avions abordée mais dont j'ai oublié la réponse : que va-t-il se passer pour les anciens articles ? Il seront inchangés ?

@thom4parisot
Copy link
Member Author

thom4parisot commented Mar 9, 2021

Exactement, les anciens articles sont inchangés : la sortie des métaonnées lors de l'export devrait donner le même résultat (je dis "devrais" au sens où c'est pas clair si on écrit les tests qui permettent de garantir cette intention).

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 a pull request may close this issue.

2 participants