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

Importer une nouvelle version d'un contenu ne mélange plus les introductions et conclusions #6350

Merged
merged 2 commits into from
Jul 11, 2022

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Jul 5, 2022

Depuis la fusion de #6203, lors de l'importation d'un contenu, l'introduction et la conclusion d'un conteneur étaient remplacés par ceux du dernier enfant de celui-ci. Cette PR corrige ce soucis et ajoute un test pour éviter une régression plus tard.

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Télécharger l'archive du tutoriel « Un zeste de Python »
  • Utiliser la fonction « Importer un nouveau contenu » avec l'archive pour créer le tutoriel
  • Utiliser la fonction « Importer une nouvelle version » avec la même archive
  • Vérifier que l'introduction et la conclusion n'a pas été modifiée
  • Vérifier que les sections not_ready_to_publish sont bien grisées comme il se doit

@Situphen Situphen added the C-Back Concerne le back-end Django label Jul 5, 2022
@Situphen Situphen changed the title Importer une nouvelle version d'un contenu ne supprime plus l'intro et la conclu Importer une nouvelle version d'un contenu ne mélange plus les introductions et conclusions Jul 5, 2022
@coveralls
Copy link

coveralls commented Jul 5, 2022

Coverage Status

Coverage remained the same at 88.033% when pulling eea2ad7 on Situphen:import into 8bcd875 on zestedesavoir:dev.

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 ✔️

@Arnaud-D Arnaud-D enabled auto-merge (squash) July 11, 2022 06:46
@Arnaud-D Arnaud-D merged commit 1bf72c7 into zestedesavoir:dev Jul 11, 2022
@Situphen Situphen deleted the import branch July 13, 2022 17:26
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
Projects
Archived in project
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants