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

Ajoute des formulaires dédiés à l'introduction et la conclusion #6642

Merged

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Sep 15, 2024

Fix #5795.

  • Simplifie le formulaire de création de publication
  • Crée un formulaire dédié à l'introduction d'une publication
  • Crée un formulaire dédié à la conclusion d'une publication
  • Supprime la fonctionnalité "Une nouvelle version a été postée avant que vous ne validiez." qui est buguée.

Le bug en question est qu'on perd totalement le contenu qu'on était en train de rédiger. C'est pire que tout, parce que si on écrase la version du serveur, on peut encore la récupérer dans l'historique, alors que la version en cours d'édition dans le navigateur, ce n'est pas le cas. J'ai crée un ticket pour suivre ce sujet : #6643.

Contrôle qualité

  • Créer un contenu
  • Modifier l'intro, tester l'aperçu et l'enregistrement (notamment dans l'historique des versions)
  • Modifier la conclusion, tester l'aperçu et l'enregistrement (notamment dans l'historique des versions)

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Sep 15, 2024
@Arnaud-D Arnaud-D changed the title Formulaires intro et conclusion Ajoute de formulaires dédiés à l'introduction et la conclusion Sep 15, 2024
@Arnaud-D Arnaud-D changed the title Ajoute de formulaires dédiés à l'introduction et la conclusion Ajoute des formulaires dédiés à l'introduction et la conclusion Sep 15, 2024
@Arnaud-D Arnaud-D force-pushed the formulaires-intro-et-conclusion branch 2 times, most recently from 888f180 to 478b757 Compare September 16, 2024 06:11
@Arnaud-D Arnaud-D force-pushed the formulaires-intro-et-conclusion branch 2 times, most recently from 112fe52 to 0b5fda7 Compare September 17, 2024 06:34
@Arnaud-D Arnaud-D force-pushed the formulaires-intro-et-conclusion branch 3 times, most recently from 507311c to 5102047 Compare September 17, 2024 21:08
@Arnaud-D Arnaud-D force-pushed the formulaires-intro-et-conclusion branch from 5102047 to b04156b Compare September 17, 2024 21:24
@coveralls
Copy link

coveralls commented Sep 18, 2024

Coverage Status

coverage: 88.938% (+0.01%) from 88.928%
when pulling f555774 on Arnaud-D:formulaires-intro-et-conclusion
into b14be34 on zestedesavoir:dev.

@Arnaud-D Arnaud-D marked this pull request as ready for review September 19, 2024 05:56
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 bien, mais j'ai deux suggestions :

  1. ajouter une ligne séparatrice (cf ma suggestion de modification)
  2. dans les formulaires pour modifier / ajouter une introduction ou une conclusion, avoir un champ pour préciser le message de suivi (quite à ce qu'il soit pré-rempli par Modification de l('intro|a conclusion)). Ça permet de préciser quelle a été la modif :)

@Arnaud-D
Copy link
Contributor Author

J'ai ajouté tes suggestions.

Je ne teste pas le message de commit dans les tests unitaires. C'est pas que c'est inutile, mais je n'ai pas vu d'exemple ailleurs, et développer un bout de code propre pour ça est une tâche à part entière. En particulier pour taper directement sur le dépôt, et ne pas utiliser la vue qui liste les commits (ce qui créerait un couplage entre vues que je veux éviter).

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

@philippemilink philippemilink enabled auto-merge (squash) September 24, 2024 19:43
@philippemilink philippemilink merged commit 1037e95 into zestedesavoir:dev Sep 24, 2024
8 checks passed
@Arnaud-D Arnaud-D deleted the formulaires-intro-et-conclusion branch September 24, 2024 20:13
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
Development

Successfully merging this pull request may close these issues.

Permettre la modification de l'introduction et conclusion d'un article "in place"
3 participants