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

Déplace la modification de la miniature dans un formulaire dédié #6613

Merged
merged 7 commits into from
Sep 15, 2024

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented May 8, 2024

Je mets le bout de formulaire qui permet de choisir une miniature/logo pour une publication dans un formulaire dédié. Ce formulaire est accessible en cliquant sur la miniature dans l'en-tête de la publication.

Fix #5797

Contrôle qualité

Tester la création de contenus

Tester l'édition de contenus (le formulaire fourre-tout "éditer")

Tester les nouvelles vues de modification de la miniature

Regarder ce qu'il se passe sur les pages de chapitres et de parties.

Regarder qu'une entrée est enregistrée dans le journal des événements du contenu.

@Arnaud-D Arnaud-D added C-Front Concerne l'interface du site C-Back Concerne le back-end Django labels May 8, 2024
@coveralls
Copy link

coveralls commented May 8, 2024

Coverage Status

coverage: 88.855% (+0.03%) from 88.821%
when pulling cda59f9 on Arnaud-D:deplace-modif-image
into d3c7fee on zestedesavoir:dev.

@Arnaud-D Arnaud-D marked this pull request as ready for review May 9, 2024 16:22
@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented May 12, 2024

Je me rends compte que j'ai oublié d'ajouter un enregistrement au journal d'événement pour cette action. Faut que je m'occupe de ça avant que ça soit prêt pour la revue.

@Arnaud-D Arnaud-D self-assigned this Jun 9, 2024
@Arnaud-D
Copy link
Contributor Author

Voilà, j'ai pu rajouter ce que je voulais.

@coveralls
Copy link

coveralls commented Jul 20, 2024

Coverage Status

coverage: 88.926% (+0.04%) from 88.887%
when pulling b34af46 on Arnaud-D:deplace-modif-image
into 29b75e0 on zestedesavoir:dev.

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.

Pas vraiment testé, et puis il faudra rebase quand #6550 sera mergée.

zds/tutorialv2/forms.py Show resolved Hide resolved
assets/scss/layout/_content.scss Outdated Show resolved Hide resolved
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.

Le commentaire à propos des images transparentes, mais sinon QA OK.

templates/tutorialv2/includes/headline/title.part.html Outdated Show resolved Hide resolved
@philippemilink philippemilink merged commit 75989e6 into zestedesavoir:dev Sep 15, 2024
12 checks passed
@Arnaud-D Arnaud-D deleted the deplace-modif-image branch September 15, 2024 19:21
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.

Modification de la miniature du contenu directement sur la page du contenu
3 participants