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

Ajout de boutons pour modifier l'introduction et la conclusion des contenus #6565

Merged
merged 4 commits into from
Mar 2, 2024

Conversation

Vayel
Copy link
Contributor

@Vayel Vayel commented Dec 29, 2023

Ajoute des boutons d'édition de l'introduction et de la conclusion des contenus.

image

Résout partiellement #5795

Contrôle qualité

Pour :

  • Une intro/conclu de tutoriel
  • Une intro/conclu de partie de tutoriel
  • Une intro/conclu d'article

vérifier :

  • si l'intro/conclu existe, vérifier la présence du bouton d'édition qui renvoie vers le champ du formulaire d'édition du contenu
  • si l'intro/conclu n'existe pas, vérifier l'absence du bouton d'édition et la présence du warning existant avant cette PR

Vérifier l'absence de ces boutons sur les versions non éditables (bêta, publiée).

@coveralls
Copy link

coveralls commented Dec 29, 2023

Coverage Status

coverage: 88.664%. remained the same
when pulling 1dc0207 on Vayel:5795_edition_intro_conclu
into 83145f1 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.

En plus des modifications ci-dessous, j'aimerais bien que cette PR aille plus loin en créant des formulaires indépendants pour la modification de l'introduction et de la conclusion. Actuellement, c'est mis en vrac dans un gros formulaire, mais ce n'est pas ce qu'on souhaite garder à l'avenir.

D'ailleurs, cette PR ne résout pas complètement le ticket que tu cites vu que dans celui-ci, il y a une grosse partie front : l'idée était d'avoir le formulaire apparaître sur la même page à la place du rendu de l'intro/conclusion.

templates/tutorialv2/view/content.html Outdated Show resolved Hide resolved
templates/tutorialv2/includes/content/content.part.html Outdated Show resolved Hide resolved
templates/tutorialv2/view/container.html Outdated Show resolved Hide resolved
@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django C-Front Concerne l'interface du site labels Dec 29, 2023
@Vayel
Copy link
Contributor Author

Vayel commented Dec 31, 2023

Merci pour les retours.

Pour le moment je laisse comme ça, peut-être que je ferai évoluer avec des formulaires dédiés.

@philippemilink philippemilink changed the title 5795 edition intro conclu Ajout de boutons pour modifier l'introduction et la conclusion des contenus Feb 24, 2024
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 ✔️

Il reste du travail pour aller au bout du sujet, mais c'est déjà un bon pas en avant. 👍

@Arnaud-D Arnaud-D enabled auto-merge (squash) March 2, 2024 17:55
@Arnaud-D Arnaud-D merged commit 6ab0afd into zestedesavoir:dev Mar 2, 2024
8 checks passed
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 C-Front Concerne l'interface du site
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants