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

N'exporte (vraiment) que les containers ready_to_publish #6282

Merged

Conversation

philippemilink
Copy link
Member

Fix #6281
Corrige edfb4ec

J'ai indenté templates/tutorialv2/export/content.md (sauf les instructions qui affichent le contenu), pour faciliter la compréhension de la logique du fichier.

Contrôle qualité

  1. Créer un billet
  2. Ajouter une section
  3. Publier le billet
  4. Exporter le billet au format Markdown, avec par exemple : python3 manage.py export_markdown ${id_contenu}
  5. Consulter le fichier Markdown généré.

Il est aussi possible de tester avec les articles, les petits tutos, les moyens tutos et les gros tutos en jouant sur les parties et chapitres à publier, mais les tests ajoutés s'assurent déjà que tout fonctionne comme attendu.

@coveralls
Copy link

coveralls commented Apr 2, 2022

Coverage Status

Coverage remained the same at 87.201% when pulling 5d08a1c on philippemilink:fix-export-ready-to-publish into ee086bd on zestedesavoir:dev.

@Situphen
Copy link
Member

Situphen commented Apr 2, 2022

Je t'avoue que je ne suis pas forcément super fan de l'indentation des blocs conditionnels dans ce cas de figure car ça donne des lignes avec deux, quatre ou six espaces. Alors ça ne doit pas changer grand chose au niveau du rendu mais je pense qu'il vaut mieux laisser ça plat.

@Situphen
Copy link
Member

Situphen commented Apr 2, 2022

Au niveau du code, les tests me semble corrects !

J'ai aussi fait quelques essais en local sur un billet et sur le tutoriel Zeste de Python et le rendu était correct.

@philippemilink philippemilink force-pushed the fix-export-ready-to-publish branch from 9fd7cae to 4ee5093 Compare April 3, 2022 14:53
@philippemilink
Copy link
Member Author

je pense qu'il vaut mieux laisser ça plat.

J'ai tout remis à plat, en ajoutant des commentaires après les balises fermantes pour indiquer à quelle balise ouvrante elles correspondent.

@philippemilink philippemilink force-pushed the fix-export-ready-to-publish branch from 4ee5093 to d7333cd Compare April 9, 2022 15:26
@philippemilink
Copy link
Member Author

J'ai aussi remis comme avant les quelques {% if foo %}{{ foo }}{% endif %}, que j'avais scindé sur trois lignes auparavant. Cette PR ne devrait maintenant plus rien changer en terme de lignes vides.

@Situphen Situphen enabled auto-merge (rebase) April 26, 2022 12:50
@Situphen Situphen merged commit 43a68ef into zestedesavoir:dev Apr 26, 2022
@philippemilink philippemilink deleted the fix-export-ready-to-publish branch May 4, 2022 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

L'export Markdown des billets ne contient pas les sections
3 participants