Projet

Général

Profil

Development #55370

api datetimes pour plusieurs agendas à la fois

Ajouté par Frédéric Péters il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
03 juillet 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision e86d0cb1 (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

api: add datetimes for multiple events agendas (#55370)

Révision fa34281a (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

api: prefetch events in multiple agendas datetimes (#55370)

Historique

#1

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Assigné à mis à Valentin Deniaud
#2

Mis à jour par Valentin Deniaud il y a plus de 2 ans

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.

#3

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)

#4

Mis à jour par Valentin Deniaud il y a plus de 2 ans

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.

#5

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.

#6

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/

#7

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.

#8

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

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

Formats disponibles : Atom PDF