zds-site icon indicating copy to clipboard operation
zds-site copied to clipboard

Ajoute un bouton pour supprimer le message d'accueil

Open philippemilink opened this issue 2 years ago • 3 comments

Cette PR ajoute un bouton pour supprimer le message d'accueil en place (actuellement, il faut vider les champs du formulaire pour supprimer le message).

QA

  1. Appliquer les migrations : make migrate
  2. Se connecter en tant qu'admin et ajouter un message d'accueil (depuis la page d'accueil : Gérer les unes > Éditer le message d'accueil)
  3. Constater que le message est affiché sur la page d'accueil
  4. Supprimer le message : aller sur la page du formulaire pour ajouter/modifier le message d'accueil et cliquer le bouton de suppression (en haut à droite ; il peut être mal aligné sur Firefox, mais ça vient d'un bug connu)
  5. Constater qu'il n'y a plus de message affiché sur la page d'accueil.

philippemilink avatar Aug 13 '22 21:08 philippemilink

Coverage Status

Coverage increased (+0.01%) to 88.216% when pulling e3400353ad08c578a701b0ade1b52a44bf3d9e15 on philippemilink:remove-featured-message into 15795fc21c9e7d26a6455a43a89f706d385baaf1 on zestedesavoir:dev.

coveralls avatar Aug 13 '22 21:08 coveralls

Ça m'embête un peu d'écrire ça après que tu aies codé cette fonctionnalité en Python. :/ Ce ne serait pas plus adapté d'ajouter simplement un script JS qui efface le contenu du formulaire à l'appui d'un bouton ?

Situphen avatar Aug 14 '22 23:08 Situphen

Ça m'embête un peu d'écrire ça après que tu aies codé cette fonctionnalité en Python. :/

:)

Ce ne serait pas plus adapté d'ajouter simplement un script JS qui efface le contenu du formulaire à l'appui d'un bouton ?

C'était aussi une possibilité (ajouter un simple bouton reset au formulaire), mais je me demandais si je n'allais pas rajouter aussi une commande de manage.py pour pouvoir supprimer le message (depuis une CRON par exemple). Si un jour on se décide à coder cette fonctionnalité, on devrait pouvoir se rebrancher sur une partie du code que j'ai rajouté.

Aussi niveau ergonomie, je trouve ça un peu bizarre de devoir vider un formulaire pour supprimer quelque chose...

philippemilink avatar Aug 15 '22 09:08 philippemilink