Développement #66326
Garde partagée, indiquer proprement le calendrier d'indispo qui contient les vacances
0%
Description
Actuellement c'est magiquement basé sur le slug. J'imagine devoir introduire un truc genre les SiteSettings de combo.
Le but de tout ça c'est aussi d'intégrer des contrôles pour empêcher la suppression du calendrier choisi.
En profiter pour avoir des on_delete=PROTECT sur les règles qui seraient menacée par une telle suppression.
Files
Associated revisions
agendas: prevent cascade deletion of custody holiday rules (#66326)
History
Updated by Valentin Deniaud over 2 years ago
- File 0001-manager-select-unavailability-calendar-containing-ho.patch 0001-manager-select-unavailability-calendar-containing-ho.patch added
- File 0002-agendas-prevent-cascade-deletion-of-custody-holiday-.patch 0002-agendas-prevent-cascade-deletion-of-custody-holiday-.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Valentin Deniaud a écrit :
Le but de tout ça c'est aussi d'intégrer des contrôles pour empêcher la suppression du calendrier choisi.
En profiter pour avoir des on_delete=PROTECT sur les règles qui seraient menacée par une telle suppression.
J'ai finalement trouvé la combinaison des deux inutiles, plutôt juste protéger ce qui est important c'est à dire les règles.
0001 pour la sélection du calendrier, 0002 pour la protection.
Updated by Lauréline Guérin over 2 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud over 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit efed292e547adf97364febe87ff2b3d2fdf4aa1d
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jun 29 11:38:33 2022 +0200
agendas: prevent cascade deletion of custody holiday rules (#66326)
commit 0c7ca56e59da73e6556488d6a41163560f7a6e3c
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Jun 28 15:17:57 2022 +0200
manager: select unavailability calendar containing holidays (#66326)Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
manager: select unavailability calendar containing holidays (#66326)