Projet

Général

Profil

Development #61065

Gestion des inscriptions - modifier les réservations à la modification d'une inscription

Ajouté par Lauréline Guérin il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
25 janvier 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision b6f7d258 (diff)
Ajouté par Lauréline Guérin il y a environ 2 ans

api: remove user bookings on subscription deletion (#61065)

Révision ad103323 (diff)
Ajouté par Lauréline Guérin il y a environ 2 ans

api: remove user bookings on subscription date changes (#61065)

Révision b6e141b9 (diff)
Ajouté par Lauréline Guérin il y a environ 2 ans

api: update user bookings extra_data on subscription update (#61065)

Historique

#2

Mis à jour par Lauréline Guérin il y a environ 2 ans

  • Description mis à jour (diff)
#3

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
  • sur un patch, paramètre update_bookings_extra_data. Si True, alors on met à jour les bookings futurs de la période de subscription.
#4

Mis à jour par Lauréline Guérin il y a environ 2 ans

  • Description mis à jour (diff)
#5

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.
#6

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.

#7

Mis à jour par Lauréline Guérin il y a environ 2 ans

  • Assigné à mis à Lauréline Guérin
#9

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 ?

#10

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

#11

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

#13

Mis à jour par Valentin Deniaud il y a environ 2 ans

  • Statut changé de Solution proposée à Solution validée
#14

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)
#15

Mis à jour par Transition automatique il y a environ 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#16

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF