Development #55369
api datetimes, options pour ne pas marquer comme désactivée une réservation de l'user_external_id
0%
Description
Actuellement si on passe à /datetimes un paramètre user_external_id, on se trouve dans le résultat avec disabled: true là où il a déjà une réservation, parce que :
def is_event_disabled(event, min_places=1): if getattr(event, 'user_places_count', 0) > 0: return True
Dans l'idée d'une mise à jour de réservations, y compris annuler une réservation existante, j'aurais souhaité que l'événement ne soit pas désactivé.
(c'est fortement en lien avec #55368)
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-api-disable-event-only-if-excluded_user_external_id-.patch 0001-api-disable-event-only-if-excluded_user_external_id-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà en introduisant une différenciation supplémentaire entre les paramètres user_external_id et exclude_user_external_id.
À noter qu'avant ce patch la seule différence c'est qu'en passant user_external_id
, on a la clé booked_for_external_user
en plus dans la réponse. Une approche alternative pourrait donc être de supprimer cette différence, déprécier exclude_user_external_id
puisqu'il deviendrait strictement équivalent à user_external_id
, et pour faire ce patch pouvoir passer un paramètre disable_booked
explicite.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-api-disable-event-only-if-excluded_user_external_id-.patch 0001-api-disable-event-only-if-excluded_user_external_id-.patch ajouté
- Statut changé de Solution validée à Solution proposée
Le patch a un peu bougé suite au rebase et j'ai aussi ajouté le comportement dans l'API de réservation multiple, je veux bien un nouveau ack.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f6de72a5a9da8e9a559962da7494db87bd2ef536 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Aug 4 10:32:41 2021 +0200 api: disable event only if excluded_user_external_id is specified (#55369)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api: disable event only if excluded_user_external_id is specified (#55369)