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

Importe l'information 'ready_to_publish' depuis les archives #6203

Merged
merged 4 commits into from
Dec 27, 2021

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 31, 2021

L'import d'archives ne prenait pas en compte l'information 'ready_to_publish', c'est chose faite. Désormais exporter un contenu et le réimporter identiquement est idempotent à ce niveau-là. Ça permet aussi de changer les 'ready_to_publish' ailleurs et réimporter sur le site, comme le font certains auteurs.

Fix #6184.

Remarque : Ce n'est pas la meilleure correction possible. L'idéal serait de refactoriser l'import des archives entièrement, mais c'est pas le même boulot de développement ni de QA. La qualité du code et des interfaces peut être pas mal améliorée sous certains aspects.

Contrôle qualité

Pour la correction du bug :

  • Créer un contenu, avec des parties sans sous parties, et des parties avec chapitre. Le but est d'avoir à la fois des conteneurs avec et sans enfants pour bien cadrer le code en testant.
  • Changer des status 'prêt à valider' dans différents endroits.
  • Exporter l'archive, constater que les statuts 'prêts à valider' sont bien ce qu'il faut.
  • Changer les statuts de manière aléatoire sur la publication
  • Importer l'archive et constater que les status de 'prêt à valider' sont mis à jour avec ceux de l'archive.

Non régression : vérifier que les autres parties de l'archive sont importées correctement (par exemple modifier l'intro, les titres, etc.).

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Oct 31, 2021
@coveralls
Copy link

coveralls commented Oct 31, 2021

Coverage Status

Coverage increased (+0.002%) to 87.02% when pulling a8fee83 on Arnaud-D:import_ready_to_publish into 5ea1809 on zestedesavoir:dev.

@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 31, 2021
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 ✔️

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 hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Prise en compte du champ ready_to_publish lors des imports
3 participants