Development #60023
[API] /api/agendas/recurring-events/fillslots/ - retourner tous les datetime effectivement bookés
0%
Description
Au retour de la réservation on a le compte des datetime bookés (_booking_count).
Mais on va souhaiter laisser dans l'historique une trace plus précise en indiquant les datetimes qui ont été bookés.
Il faudrait les ajouter au retour comme on ajoute les datetimes non bookés dans (_full_events).
Ceci pourrait être paramétré dans l'appel.
En effet il se pourrait que dans le cas d'usage famille il y ait inscription à 10 événements sur 30 semaines ce qui remonterait 300 bookings et ne serait pas exploitable. Dans mon cas ce serait plutôt une quinzaine et ce serait très bien.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Mikaël Ates a écrit :
Avec une subtilité que tu n'as peut-être pas remarqué :Au retour de la réservation on a le compte des datetime bookés (_booking_count).
- Réservation partielle pour 1 mois d'un évènement tous les jours (via date_start et date_end) -> booking_count vaut 4
- Extension de cette réservation pour 1 mois de plus -> booking_count vaut 4, alors qu'il y a actuellement 8 réservations en cours
Tu voudrais quoi dans le retour, les 4 évènements effectivement réservés par l'appel ou le total des réservations futures ?
Mis à jour par Mikaël Ates il y a plus de 2 ans
Tu voudrais quoi dans le retour, les 4 évènements effectivement réservés par l'appel
Oui, ces 4 événements car c'est le résultat de cet appel.
Pour retrouver tous les bookings d'un user sur un événement c'est #60024.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-api-add-option-to-return-booked-events-detail-in-fil.patch 0001-api-add-option-to-return-booked-events-detail-in-fil.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
C'est important de conditionner ça au paramètre include_booked_events_detail ?
Vu de loin et par souci de simplicité j'aurais renvoyé les infos systématiquement.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Emmanuel Cazenave a écrit :
C'est important de conditionner ça au paramètre
include_booked_events_detail?
Ouep, dernière phrase de la description :
Mikaël Ates a écrit :
En effet il se pourrait que dans le cas d'usage famille il y ait inscription à 10 événements sur 30 semaines ce qui remonterait 300 bookings et ne serait pas exploitable. Dans mon cas ce serait plutôt une quinzaine et ce serait très bien.
Mis à jour par Emmanuel Cazenave 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-add-option-to-return-booked-events-detail-in-fil.patch 0001-api-add-option-to-return-booked-events-detail-in-fil.patch ajouté
- Statut changé de Solution validée à Solution proposée
Valentin Deniaud a écrit :
Emmanuel Cazenave a écrit :
C'est important de conditionner ça au paramètre
include_booked_events_detail?Ouep, dernière phrase de la description :
Mikaël Ates a écrit :
En effet il se pourrait que dans le cas d'usage famille il y ait inscription à 10 événements sur 30 semaines ce qui remonterait 300 bookings et ne serait pas exploitable. Dans mon cas ce serait plutôt une quinzaine et ce serait très bien.
Et en relisant mon patch je me suis rappelé que pour la cohérence j'avais aussi ajouté la clé booked_events dans le retour des API de réservation d'évènements multiples : ici pas besoin de conditionner avec le paramètre include_booked_events_detail, effectivement. Je veux bien un nouveau ack :)
Mis à jour par Emmanuel Cazenave 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 3ace9e4505b453c1e093acb36461c974a5bc9015
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Dec 23 15:07:29 2021 +0100
api: add option to return booked events detail in fillslots APIs (#60023)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
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Lié à Development #63044: API fillslots multiple & recurring: renvoyer le détail des réservations créées ajouté
api: add option to return booked events detail in fillslots APIs (#60023)