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

Rend possible la mise à jour d'image avec un SVG #6207

Merged
merged 3 commits into from
Nov 18, 2021

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Nov 16, 2021

  • Rend possible la mise à jour d'image avec un SVG
  • Ajoute des tests pour les images SVG (pour éviter que le soucis ne se reproduise)
  • Nettoyage des tests des vues des galeries (tant qu'à faire)

Sujet du forum où le bug a été reporté

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Aller dans une galerie
  • Aller sur une image
  • Sélectionner une image SVG (ne pas choisir une image trop complexe)
  • Envoyer le formulaire (avec le bouton « Mettre à jour »)
  • Vérifier que l'image s'affiche correctement

Bien veiller à utiliser le bouton Rebase and merge pour garder les 3 commits atomiques.

@Situphen Situphen added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Nov 16, 2021
@coveralls
Copy link

coveralls commented Nov 16, 2021

Coverage Status

Coverage decreased (-0.004%) to 86.729% when pulling d7c7dab on Situphen:update-svg into 2d8d3ca on zestedesavoir:dev.

@Situphen Situphen marked this pull request as ready for review November 16, 2021 21:18
@Situphen
Copy link
Member Author

J'ai effectué les corrections demandées ! J'ai déplacé ce bout de code qui vérifie que l'image et les miniatures ont bien été supprimées dans le test qui vérifie la suppression des images (car actuellement c'est dans le test qui vérifie la mise à jour d'une image donc ça n'a rien à faire là) :

# picture AND thumbnails should be gone
self.assertEqual(nb_files, len(os.listdir(self.gallery1.get_gallery_path())))

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Rapport de QA

OK ✔️

@philippemilink philippemilink merged commit 1c8e00b into zestedesavoir:dev Nov 18, 2021
@Situphen Situphen deleted the update-svg branch November 18, 2021 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants