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

Affiche les images dans les ePUBs #6357

Merged
merged 5 commits into from Jul 24, 2022

Conversation

philippemilink
Copy link
Member

Fix #6319

Cette PR corrige l'affichage des images dans les exports ePUBs et fait un peu de nettoyage de code. J'en ai profité pour corriger un problème qui empêche l'affichage de n'importe quelle image dans les ePUBs si zds-site est cloné et fonctionne dans un dossier dont le chemin absolu commence par settings.MEDIA_URL (/media/ en l'occurrence -- oui, chez moi c'est /media/linux/dev/zds/zds-site...)

Contrôle qualité

  1. Créer un billet avec un plusieurs images avec des URLs de différents types :
    • une URL relative, par exemple en glissant-déposant l'image dans le nouvel éditeur (donnera une URL du type /media/...)
    • une URL absolue, par exemple ajouter une image dans la galerie du contenu et utiliser son URL absolue (du style `http://localhost:8000/media/...)
  2. Publier le billet
  3. Exporter le billet en ePUB : python manage.py generate_epub <id du billet>
  4. Ouvrir le fichier ePUB (dans contents-public/<slug du villet>/extra_contents) : les images doivent bien s'afficher
  5. Regarder le contenu du dossier utilisé pour construire le ePUB (par exemple tree contents-public/<slug du villet>/extra_contents) : il ne doit pas y avoir d'autres images que celles présentes dans le contenu (et sprite.png)
  6. Exporter à nouveau le billet : python manage.py generate_epub <id du billet>
  7. Répéter les étapes 4 et 5 : les noms des images ont changé, mais il n'y pas deux fois la même image.

Optionnel : tout refaire, mais avec zds-site cloné dans un dossier dont le chemin absolu commence par /media/.

@philippemilink philippemilink added this to Open PR in Objectif v30.6 via automation Jul 17, 2022
@philippemilink philippemilink added this to En développement in Suivi des PR via automation Jul 17, 2022
@coveralls
Copy link

coveralls commented Jul 17, 2022

Coverage Status

Coverage increased (+0.003%) to 88.187% when pulling 6920fb2 on philippemilink:fix-image-epubs into 5eddc4b on zestedesavoir:dev.

@philippemilink philippemilink moved this from En développement to En attente de QA in Suivi des PR Jul 17, 2022
@philippemilink philippemilink added S-BUG Corrige un problème C-Epub Concerne l'export en epub labels Jul 17, 2022
Copy link
Member

@Situphen Situphen 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 Ça fonctionne bien chez moi !

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Jul 24, 2022
@Situphen Situphen enabled auto-merge (rebase) July 24, 2022 10:17
@Situphen Situphen merged commit b4d2d08 into zestedesavoir:dev Jul 24, 2022
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Jul 24, 2022
Objectif v30.6 automation moved this from Open PR to Done Jul 24, 2022
@philippemilink philippemilink deleted the fix-image-epubs branch July 24, 2022 11:39
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
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

Les images n'apparaissent pas dans les ePUBs
3 participants