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

Corrige le contenu des ePUBs pour interpréter l'HTML #6318

Merged
merged 2 commits into from
Jun 6, 2022

Conversation

philippemilink
Copy link
Member

Fix #6313

Cette PR permet de ne pas échapper les balises HTML lorsque les ePUBs sont générés (et ajoute un test minimal)

Contrôle qualité

  • Créer un billet (plus facile à publier), avec un peu de mise en forme (du gras, souligné, etc, pour que ça génère des balises HTML) ; le publier ;
  • Générer l'export en ePUB : python manage.py generate_epub $id_du_contenu
  • Ouvrir le fichier ePUB généré (j'utilise personnellement ebook-viewer de Calibre) et s'assurer que les balises HTML ne sont pas apparentes, mais bien interprétées.

Et merci @artragis pour la solution au bug ! :)

@philippemilink philippemilink added S-BUG Corrige un problème C-Epub Concerne l'export en epub labels May 24, 2022
@coveralls
Copy link

coveralls commented May 24, 2022

Coverage Status

Coverage increased (+0.09%) to 87.784% when pulling 1aabb89 on philippemilink:fix-epub-html into cb575dc on zestedesavoir:dev.

@Arnaud-D
Copy link
Contributor

Arnaud-D commented Jun 3, 2022

Je ne suis pas sûr de [ce] que j'ai vu en testant.

J'ai quand même un doute sur quelque chose, cependant. J'ai voulu tester en redemandant l'export depuis l'interface sur un contenu déjà publié avant de faire tes instructions de QA, mais ça ne marchait pas. Après avoir fait ta commande, plus de soucis à ce niveau-là. L'export se base sur des choses mises en cache d'une certaine manière ?

@philippemilink
Copy link
Member Author

Je ne suis pas sûr de que j'ai vu en testant.

Il manque un mot ;)

J'ai quand même un doute sur quelque chose, cependant. J'ai voulu tester en redemandant l'export depuis l'interface sur un contenu déjà publié avant de faire tes instructions de QA, mais ça ne marchait pas. Après avoir fait ta commande, plus de soucis à ce niveau-là. L'export se base sur des choses mises en cache d'une certaine manière ?

Ce que je viens de faire :

  • depuis la branche dev, création et publication d'un billet avec un peu de HTML dedans. Demande des exports depuis la page web : le ePub généré affiche les balises HTML
  • bascule sur la branche de la PR, nouvelle demande de génération des exports (depuis la page web) : le ePub généré a ses balises HTML correctement interprétées et non échappées.

Donc ça semble fonctionner chez moi.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

Je n'avais pas relancé le watcher en changeant de branche l'autre fois, donc il tournait avec l'ancien code. Ça fonctionne bien.

@Arnaud-D Arnaud-D enabled auto-merge (squash) June 6, 2022 14:19
@Arnaud-D Arnaud-D merged commit fa390e3 into zestedesavoir:dev Jun 6, 2022
@philippemilink philippemilink deleted the fix-epub-html branch June 6, 2022 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Epub Concerne l'export en epub S-BUG Corrige un problème
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Le code HTML des ePUBs est visible
3 participants