Development #61065
Gestion des inscriptions - modifier les réservations à la modification d'une inscription
0%
Description
Lorsqu'on modifie les dates d'une inscription (pour raccourcir la période) ou qu'on supprime l'inscription, il faudrait annuler (ou supprimer ?) les réservations posées qui étaient couvertes par la période initiale et qui ne matchent plus aucune inscription du user
Il faudrait permettre de modifier les extra datas des réservations futures (même les annulées ? si on les affiche dans la page de pointage ?), pour par exemple mettre à jour la classe d'un enfant sur les réservations déjà posées.
Fichiers
Révisions associées
api: remove user bookings on subscription date changes (#61065)
api: update user bookings extra_data on subscription update (#61065)
Historique
Mis à jour par Lauréline Guérin il y a environ 2 ans
Ajout d'un paramètre ?
Proposition:- sur un delete ou un patch, paramètre
outdated_bookings_action
- si =
cancel
, alors les bookings qui étaient couverts par la période initiale et qui ne matchent plus aucune inscription du user sont annulés - si =
delete
, alors les bookings qui étaient couverts par la période initiale et qui ne matchent plus aucune inscription du user sont supprimés - sinon, rien
- si =
- sur un patch, paramètre
update_bookings_extra_data
. Si True, alors on met à jour les bookings futurs de la période de subscription.
Mis à jour par Valentin Deniaud il y a environ 2 ans
Lauréline Guerin a écrit :
Ajout d'un paramètre ?
Si il n'y a pas deux cas d'usages identifiés pour un coup annuler, un coup supprimer, je pense qu'il faut imposer un comportement par défaut et puis voilà, pareil pour la mise à jour des extra_data.
Le choix entre les deux est plus fonctionnel que technique je pense :- Si on supprime, il faut faire attention côté demande à ce que l'usager ne puisse pas se servir de la modification d'une inscription pour effectuer une annulation de dernière minute de manière silencieuse, qui devrait lui être facturée.
- Si on annule, on risque d'encombrer toutes les vues qui listent les inscriptions annulées. Notamment, si les familles touchent un peu à tout au début de l'année et provoquent des centaines d'annulation autour de la fin de l'année en ajustant la date de fin de leur inscription, alors une fois en fin d'année il sera vraiment peu utile d'avoir trace de ça.
Mis à jour par Stéphane Guiet il y a environ 2 ans
Pour ce cas d'usage, soit la modification des dates d'inscription (changement de la date de fin)
il est bien question de supprimer les bookings postérieurs à cette nouvelle date de fin.
Noter que ce changement de date n'est pas à la main des parents : nous sommes dans un cas 100% back-office donc pas de risque de contournement comme cité.
Tout au plus le parent peut demander un changement de site pour une activité, ce qui aurait pour conséquence indirecte de clore l'inscription : l'agent instruit et décide de la date de fin.
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Fichier 0003-api-update-user-bookings-extra_data-on-subscription-.patch 0003-api-update-user-bookings-extra_data-on-subscription-.patch ajouté
- Fichier 0002-api-remove-user-bookings-on-subscription-date-change.patch 0002-api-remove-user-bookings-on-subscription-date-change.patch ajouté
- Fichier 0001-api-remove-user-bookings-on-subscription-deletion-61.patch 0001-api-remove-user-bookings-on-subscription-deletion-61.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a environ 2 ans
Je crois qu'il n'y a pas de cas d'usage légitime justifiant la création d'inscriptions qui se chevauchent. Est-ce qu'on ne gagnerait pas en simplicité pour ici et pour l'avenir à interdire la création de telles inscriptions ?
Mis à jour par Stéphane Guiet il y a environ 2 ans
Je ne vois pas de cas ou deux inscriptions à un même agenda pourraient se recouvrir
Mis à jour par Lauréline Guérin il y a environ 2 ans
ok, j'ai fait #61631, je rebase ce ticket dessus et je simplifie les tests
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Fichier 0005-api-update-user-bookings-extra_data-on-subscription-.patch 0005-api-update-user-bookings-extra_data-on-subscription-.patch ajouté
- Fichier 0004-api-remove-user-bookings-on-subscription-date-change.patch 0004-api-remove-user-bookings-on-subscription-date-change.patch ajouté
- Fichier 0003-api-remove-user-bookings-on-subscription-deletion-61.patch 0003-api-remove-user-bookings-on-subscription-deletion-61.patch ajouté
Mis à jour par Valentin Deniaud il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit b6e141b9c34e6918f9609e251d669bce8ad4589f Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Thu Feb 3 16:50:21 2022 +0100 api: update user bookings extra_data on subscription update (#61065) commit ad103323ba1d1e3860666f1fc2f4dd5c6c47601a Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Thu Feb 3 15:37:09 2022 +0100 api: remove user bookings on subscription date changes (#61065) commit b6f7d25808b35e228a03f0294c1cd04410bcae70 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Thu Feb 3 09:45:19 2022 +0100 api: remove user bookings on subscription deletion (#61065)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api: remove user bookings on subscription deletion (#61065)