Development #28827
Planitech round 2
0%
Description
Pardon pour ce gros patch, mais des changements motivés par des considérations ergonomiques, que je n'ai pas réussis à mener autrement qu'en faisant un gros lifting du connecteur.
On voudrait que l'utilisateur ait une vision globale de la disponibilité des salles et de de leurs caractéristiques avant d'avoir à choisir une date ou une salle.
Je fusionne les deux endpointgetdate et getplace en un seul getfreegaps, qui reprend le nom du endpoint utilisé coté planitech.Je lui ajoute des paramètres facilitant son utilisation : plutôt que de demander des dates précises on peut spécifier des nombres de jours qui seront interprétés comme des delta par rapport à now.
Pour ce endpoint un nouveau paramètre
display, qui accepte les valeurs 'date', 'place' et 'full'.
- 'date' : renvoie les dates répondant aux critères, au format data source
- 'place': renvoie les salles correspondant aux critères, au format data source
- 'full': renvoie les deux choses précédentes plus une structure de données qui est pratique pour un affichage global des disponibilités des salles en fonction des critères (voir capture d'écran).
Aussi enrichissement du référentiel des salles par leur adresse.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Fichier 0001-create-getfreegaps-enpoint-fot-availibilty-queries-2.patch 0001-create-getfreegaps-enpoint-fot-availibilty-queries-2.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
J'ai mis en priorité haute parce qu'une démo est attendue Jeudi.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Fichier 0001-create-getfreegaps-enpoint-fot-availibilty-queries-2.patch 0001-create-getfreegaps-enpoint-fot-availibilty-queries-2.patch ajouté
Voilà, avec aussi l'ajout de 'end_days' dans la description des paramètres du endpoint, qui avait été oublié.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Ok (pas testé le moins du monde, pas même exécuté les tests). (curieux qu'il y ait quelques "" pour les chaines, dans une grosse majorité de '', sans raison apparente).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fce83c8a3a0804f1edc22687ac52b5373311babe
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Mon Dec 10 19:05:07 2018 +0100
create 'getfreegaps' enpoint fot availibilty queries (#28827)
It has the same capabilities than 'getdays' by using display=date
and 'getplaces' by using display=place.
Search intervals can be specificed with days delta from now.
display=full will return a data easy to use in a template to display
global availability.
The place referential gain adresses.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Frédéric Péters a écrit :
Ok (pas testé le moins du monde, pas même exécuté les tests). (curieux qu'il y ait quelques "" pour les chaines, dans une grosse majorité de '', sans raison apparente).
J'utilise les simple et double quote de façon indifférenciée sans aucune raison. Quelle est la bonne pratique ?
Mis à jour par Frédéric Péters il y a plus de 5 ans
Une utilisation uniforme rend le tout plus agréable. En gros majoritairement ici des '' je resterais là-dessus partout. (avec en exception les chaines contenant des apostrophes).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution déployée à Fermé
create 'getfreegaps' enpoint fot availibilty queries (#28827)
It has the same capabilities than 'getdays' by using display=date
and 'getplaces' by using display=place.
Search intervals can be specificed with days delta from now.
display=full will return a data easy to use in a template to display
global availability.
The place referential gain adresses.