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

L'export Markdown des billets ne contient pas les sections #6281

Closed
philippemilink opened this issue Mar 31, 2022 · 2 comments · Fixed by #6282
Closed

L'export Markdown des billets ne contient pas les sections #6281

philippemilink opened this issue Mar 31, 2022 · 2 comments · Fixed by #6282
Assignees
Labels
Bloquant Ticket qui doit être traité avant la prochaine mise à jour C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant

Comments

@philippemilink
Copy link
Member

Description du bug

L'export Markdown des billets ne contient pas les sections des billets, seulement l'introduction et la conclusion. Très certainement lié à #6266.

Comment reproduire ?

La liste des étapes qui permet de reproduire le bug :

  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é.
@philippemilink philippemilink added S-BUG Corrige un problème S-Régression Corrige un problème sur un composant qui fonctionnait auparavant C-Back Concerne le back-end Django Bloquant Ticket qui doit être traité avant la prochaine mise à jour labels Mar 31, 2022
@philippemilink
Copy link
Member Author

Si quelqu'un souhaite s'attaquer au bug, je joins le patch qui contient déjà le test correspondant (c'est comme ça plus facile de tester la correction du bug ; il n'est pas complet et demande peu de refacto, mais c'est déjà un début pour tester facilement).

@Situphen
Copy link
Member

Oups ! Je crois que la solution est d'effectuer ces modifications :

---{% if child.ready_to_publish %}
+++{% if content.has_extracts or child.ready_to_publish %}
---{% if subchild.ready_to_publish %}
+++{% if child.has_extracts or subchild.ready_to_publish %}

Car soit il s'agit d'extraits (dernier niveau de conteneurs) et dans ce cas-là il n'y a pas d'étiquette ready_to_publish, soit il s'agit de conteneurs avec des sous-conteneurs et il y a l'étiquette ready_to_publish.

@Situphen Situphen removed the S-BUG Corrige un problème label Apr 1, 2022
@philippemilink philippemilink self-assigned this Apr 2, 2022
philippemilink added a commit to philippemilink/zds-site that referenced this issue Apr 2, 2022
philippemilink added a commit to philippemilink/zds-site that referenced this issue Apr 3, 2022
philippemilink added a commit to philippemilink/zds-site that referenced this issue Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bloquant Ticket qui doit être traité avant la prochaine mise à jour C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants