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.

Sign up for GitHub

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

Jump to bottom

Ajoute la modification du titre et du sous-titre d'une publication depuis une modale #6590

Merged
merged 15 commits into from Mar 31, 2024
Merged

Ajoute la modification du titre et du sous-titre d'une publication depuis une modale #6590

merged 15 commits into from Mar 31, 2024

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Mar 6, 2024 •
edited

Fix #5796.

C'est une reprise sous une forme différente d'une ancienne PR pour résoudre ce ticket.

  • Change l'agencement de la ligne de titre et sous-titre pour accomoder deux boutons d'ouverture des modales
  • Déplace au passage la licence avec les autres métainformations (sa place n'avait pas de logique particulière en vérité)
  • Ajoute le code nécessaire pour traiter ces deux nouveaux formulaires dans les modales
  • Modifie le formulaire "éditer" d'un contenu pour enlever la modification du titre et sous-titre qui s'y trouvaient jusqu'à présent
  • Le formulaire de création de contenu ne change pas pour le moment (on garde la possibilité d'y saisir titre et sous-titre.

image

Les commits sont découpés au mieux pour faciliter la revue.

Contrôle qualité

Tester la modification du titre et sous-titre sur la page de brouillon. Vérifier notamment que la date de mise à jour est modifiée et qu'on a bien un commit dans l'historique des versions.

Voir si l'affichage des boutons est absent depuis la page en ligne, celle de validation et celle de bêta.

Vérifier que le formulaire "éditer" fonctionne toujours bien.

@Arnaud-D Arnaud-D added C-Front Concerne l'interface du site C-Back Concerne le back-end Django labels Mar 6, 2024
@Arnaud-D Arnaud-D added this to En développement in Suivi des PR via automation Mar 6, 2024
@Arnaud-D Arnaud-D force-pushed the titre-in-place branch 3 times, most recently from ee4df8d to 0a2f2f0 Compare March 7, 2024 19:44
@coveralls
Copy link

coveralls commented Mar 7, 2024 •
edited

Coverage Status

coverage: 88.752% (+0.05%) from 88.699%
when pulling 031ff07 on Arnaud-D:titre-in-place
into 6e0d801 on zestedesavoir:dev.

@Arnaud-D Arnaud-D force-pushed the titre-in-place branch 4 times, most recently from 78e34c1 to 68f6fe4 Compare March 9, 2024 09:42
@Arnaud-D Arnaud-D marked this pull request as ready for review March 9, 2024 09:48
@Arnaud-D Arnaud-D moved this from En développement to En attente de QA in Suivi des PR Mar 9, 2024
@Arnaud-D
Ajoute le design du formulaire de titre et sous-titre
41454ba
@Arnaud-D
Ajoute le formulaire de mise à jour du titre
c6d6c4a
@Arnaud-D
Ajoute le formulaire de mise à jour du sous-titre
638ea5d
@Arnaud-D
Modifie le formulaire "Éditer" pour en retirer le titre et la descrip…
e10cdf8 
…tion
@Arnaud-D
Rétablit les logs pour les changements de slug
0766cf7
@Arnaud-D
Déplace l'encart de licence dans les métadonnées
edc48c9
@Arnaud-D
Corrige le test test_republish_with_different_slug
97373a0
@Arnaud-D
Corrige le test test_no_bad_slug_renaming_on_rename
be51801
@Arnaud-D
Rétablit la mise à jour du titre d'une galerie lors d'une modificatio…
e4c1981 
…n du titre d'une publication (+ refacto)
@Arnaud-D
Corrige le test test_basic_tutorial_workflow
cc84db4
@Arnaud-D
Corrige le test test_publish_content_change_title_before_watchdog
547c740
@Arnaud-D
Corrige le test test_history_navigation
7e6630f
@Arnaud-D
Ajoute les tests pour la modification du titre
86f97be
@Arnaud-D
Ajoute les tests pour la modification du sous-titre
076291c
Suivi des PR automation moved this from En attente de QA to Modification demandée Mar 30, 2024
philippemilink
philippemilink requested changes Mar 30, 2024
View reviewed changes
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.

Ça fonctionne, juste une petite question/suggestion dans le code.

zds/tutorialv2/forms.py Outdated Show resolved Hide resolved
@Arnaud-D Arnaud-D moved this from Modification demandée to En attente de QA in Suivi des PR Mar 30, 2024
philippemilink
philippemilink approved these changes Mar 31, 2024
View reviewed changes
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.

QA OK ✔️

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Mar 31, 2024
@philippemilink philippemilink merged commit c232671 into zestedesavoir:dev Mar 31, 2024
12 checks passed
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Mar 31, 2024
@Arnaud-D Arnaud-D deleted the titre-in-place branch April 1, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippemilink philippemilink philippemilink approved these changes

Assignees
No one assigned
Labels
C-Back Concerne le back-end Django C-Front Concerne l'interface du site
Projects
Suivi des PR
  
Fusionnée
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Modification du titre et du sous-titre d'un contenu "in place"
3 participants
@Arnaud-D @coveralls @philippemilink