Fix : pouvoir supprimer le dernier créneau d'un bucket

[raphodn]

Quoi ?

Actuellement, lorsqu'on affiche un "bucket" (groupe de créneaux à une même date/heure donnée), on peut effectuer des actions en AJAX sans rafraichir la page (grâce à #674)
Mais lorsqu'il reste qu'1 seul créneau, le supprimer revient à supprimer le bucket. Mais cela ne se produit pas, et il y a un bug. Pour l'éviter, j'ai disable le bouton supprimer du seul créneau libre, pour pousser à la suppression du bucket directement.

Captures d'écran

Après

[petitalb]

Je ne suis pas sur de comprendre le choix :

Mais lorsqu'il reste qu'1 seul créneau, le supprimer revient à supprimer le bucket. Mais cela ne se produit pas, et il y a un bug.

Pourquoi ne pas corriger le bug plutot que de désactiver le bouton ?

[raphodn]

Pourquoi ne pas corriger le bug plutot que de désactiver le bouton ?

J'ai essayé rapidement et je n'ai pas réussi (ca demande de modifier le controller + supprimer la modale dans le cas particulier où il n'y a qu'1 seul shift dans le bucket)

Je me dis aussi que c'est plus clair pour l'utilisateur, 1 bucket doit avoir au moins 1 shift, si on veut supprimer le shift alors ça revient à supprimer le bucket ? Vs avoir 2 boutons "supprimer" qui font la même chose in fine. (Et dans ce cas là il faudrait enlever le bouton d'ailleurs)

[petitalb]

Pourquoi ne pas corriger le bug plutot que de désactiver le bouton ?

J'ai essayé rapidement et je n'ai pas réussi (ca demande de modifier le controller + supprimer la modale dans le cas particulier où il n'y a qu'1 seul shift dans le bucket)

Je me dis aussi que c'est plus clair pour l'utilisateur, 1 bucket doit avoir au moins 1 shift, si on veut supprimer le shift alors ça revient à supprimer le bucket ? Vs avoir 2 boutons "supprimer" qui font la même chose in fine. (Et dans ce cas là il faudrait enlever le bouton d'ailleurs)

J'ai corrigé le bug !