Project

General

Profile

Development #57609

Interdire de rendre un évènement récurrent si il a déjà des réservations

Added by Valentin Deniaud 20 days ago. Updated 7 days ago.

Status:
Solution proposée
Priority:
Normal
Category:
-
Target version:
-
Start date:
05 Oct 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Pour ne pas créer un monstre.


Files

History

#1

Updated by Valentin Deniaud 20 days ago

Il s'agit juste de désactiver le champ. On pourrait vouloir mettre un help_text pour dire pourquoi, mais comme il va d'un coup s'afficher partout et qu'il n'apportera pas d'info utile dans la majorité des cas, plutôt non.

#2

Updated by Lauréline Guerin 20 days ago

et si la réservation a été annulée ?

#3

Updated by Valentin Deniaud 20 days ago

Lauréline Guerin a écrit :

et si la réservation a été annulée ?

Mini-monstre mais monstre quand même, un évènement récurrent ne doit pas avoir de réservation tout court, je trouve ça plus rassurant.

#4

Updated by Benjamin Dauvergne 14 days ago

mais comme il va d'un coup s'afficher partout et qu'il n'apportera pas d'info utile dans la majorité des cas, plutôt non.

Je n'ai pas compris ce commentaire, de la même manière que du modifies .disabled tu pourrais modifier le .help_text.

#5

Updated by Benjamin Dauvergne 14 days ago

  • Status changed from Solution proposée to En cours
#6

Updated by Valentin Deniaud 14 days ago

Benjamin Dauvergne a écrit :

mais comme il va d'un coup s'afficher partout et qu'il n'apportera pas d'info utile dans la majorité des cas, plutôt non.

Je n'ai pas compris ce commentaire, de la même manière que du modifies .disabled tu pourrais modifier le .help_text.

Oui ma remarque n'était pas d'ordre technique. C'est juste que si je modifie le help_text, alors sur les agendas events qui n'utilisent pas les évènements récurrents, donc 99%, quand tu édites un évènement qui a des réservations tu vas avoir un texte qui te dit que attention il ne faut pas que tu puisses rendre cet évènement récurrent, alors que ça ne t'es de toute façon jamais venu à l'idée. D'où « l'info est inutile dans la majorité des cas ».
Mais pour couvrir le cas où quelqu'un se demanderait pourquoi la case est grisée, je peux mettre l'info dans un attribut title peut-être ?

#7

Updated by Benjamin Dauvergne 11 days ago

Valentin Deniaud a écrit :

Oui ma remarque n'était pas d'ordre technique. C'est juste que si je modifie le help_text, alors sur les agendas events qui n'utilisent pas les évènements récurrents, donc 99%, quand tu édites un évènement qui a des réservations tu vas avoir un texte qui te dit que attention il ne faut pas que tu puisses rendre cet évènement récurrent, alors que ça ne t'es de toute façon jamais venu à l'idée. D'où « l'info est inutile dans la majorité des cas ».
Mais pour couvrir le cas où quelqu'un se demanderait pourquoi la case est grisée, je peux mettre l'info dans un attribut title peut-être ?

Ok je comprends mieux, alors je mettrais plutôt le texte d'aide uniquement quand ça reste modifiable sous forme d'alerte "Attention le choix de la récurrence deviendra définitif/non-modifiable après la première réservation." et sinon title aussi si tu veux, mais ça me parait mieux de prévenir avant qu'après.

#8

Updated by Valentin Deniaud 7 days ago

Voilà j'ai fait comme tu dis.

Also available in: Atom PDF