Development #55370
api datetimes pour plusieurs agendas à la fois
0%
Description
On voudrait présenter une page avec les événements restauration scolaire et périscolaire réunis, il n'existe pas de regroupement fixe, c'est dans la démarche que se déterminent les agendas à afficher (en fonction de l'enfant choisi on déterminera qu'il faut les agendas restauration-scolaire-anatole-france et periscolaire-jean-moulin).
Il faudrait un appel /datetimes qui puisse ainsi retourner les événements de plusieurs agendas.
À réfléchir aussi ce que ça signifie derrière en terme d'API "fillslots" pour la réservation multiple (ça veut sans doute dire inclure dans l'id des événements une référence à l'agenda).
Fichiers
Révisions associées
api: prefetch events in multiple agendas datetimes (#55370)
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0002-api-prefetch-events-in-multiple-agendas-datetimes-55.patch 0002-api-prefetch-events-in-multiple-agendas-datetimes-55.patch ajouté
- Fichier 0001-api-add-datetimes-for-multiple-events-agendas-55370.patch 0001-api-add-datetimes-for-multiple-events-agendas-55370.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Frédéric Péters a écrit :
À réfléchir aussi ce que ça signifie derrière en terme d'API "fillslots" pour la réservation multiple
Yep mais dans un autre ticket (#56104), les deux ne vont pas forcément de pair vu que là on peut déjà réserver un évènement unique avec le fillslot_url associé.
(ça veut sans doute dire inclure dans l'id des événements une référence à l'agenda).
Ça par contre oui de toute façon, pour ne pas se retrouver avec des ids qui soient les mêmes.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
note: ça serait sans doute pratique d'avoir en sortie les créneaux ordonnées par (date, agenda) (où l'ordre des agendas est celui des slugs passés en param)
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0002-api-prefetch-events-in-multiple-agendas-datetimes-55.patch 0002-api-prefetch-events-in-multiple-agendas-datetimes-55.patch ajouté
- Fichier 0001-api-add-datetimes-for-multiple-events-agendas-55370.patch 0001-api-add-datetimes-for-multiple-events-agendas-55370.patch ajouté
Lauréline Guerin a écrit :
note: ça serait sans doute pratique d'avoir en sortie les créneaux ordonnées par (date, agenda) (où l'ordre des agendas est celui des slugs passés en param)
Yep, j'ai ajouté ça.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
À noter, je ne sais pas si c'est important
urlpatterns = [ url(r'^agenda/$', views.agendas), + url(r'^agenda/datetimes/$', views.agendas_datetimes, name='api-agendas-datetimes'), url(r'^agenda/(?P<agenda_identifier>[\w-]+)/$', views.agenda_detail), url(r'^agenda/(?P<agenda_identifier>[\w-]+)/datetimes/$', views.datetimes, name='api-agenda-datetimes'),
Ça empêche un agenda d'avoir « datetimes » pour slug. On pourrait choisir de nommer l'api /agendaS/datetimes/ pour ne pas avoir ce potentiel problème.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Ça empêche un agenda d'avoir « datetimes » pour slug. On pourrait choisir de nommer l'api /agendaS/datetimes/ pour ne pas avoir ce potentiel problème.
Bien vu; ça serait plus prudent en effet, et pas illogique de mettre un 's' à @/agendas/datetimes/
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Lauréline Guerin a écrit :
Bien vu; ça serait plus prudent en effet, et pas illogique de mettre un 's' à @/agendas/datetimes/
Hop, fait sur la branche.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fa34281ade40f520542989af6fef33246f629bbf Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu Aug 5 17:17:34 2021 +0200 api: prefetch events in multiple agendas datetimes (#55370) commit e86d0cb11f054de6de90d52da8ab461993d43366 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Aug 9 11:54:18 2021 +0200 api: add datetimes for multiple events agendas (#55370)
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: add datetimes for multiple events agendas (#55370)