Development #49174
api statistiques: adaptations pour combo
0%
Description
- Il y a un paramètre time_interval requis, mais combo n'a aucun moyen de le savoir. On avait déjà prévu un système pour annoncer les filtres dispo, réutilisons ça, et mettons une valeur par défaut au paramètre au cas où il ne soit pas spécifié.
- Erreur si 'service' et 'ou' sont précisés en même temps, pas envie d'avoir à afficher ce genre d'erreur dans combo, à la place si on reçoit les deux on ignore 'ou' et ça fonctionne très bien.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-api-small-adjustements-to-statistics-endpoints-49174.patch 0001-api-small-adjustements-to-statistics-endpoints-49174.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a plus de 3 ans
J’arrive pas à capter ce qui justifie la disparition de l’intervalle de temps défini à partir d’une horodate (timestamp) ? J’avoue qu’à vue de nez j’arrive pas à voir où le timestamp casserait le reste du patch. Je loupe un truc ?
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Paul Marillonnet a écrit :
J’arrive pas à capter ce qui justifie la disparition de l’intervalle de temps défini à partir d’une horodate (timestamp) ? J’avoue qu’à vue de nez j’arrive pas à voir où le timestamp casserait le reste du patch. Je loupe un truc ?
Yep, oublié de mentionner ça dans la description, pour l'instant on a pas du tout envie de permettre de récupérer les données sans regroupement, ça risque de ne pas passer dans les tuyaux et l'affichage dans combo sera moche. Ça pourra être réactivé quand on en aura besoin, en ajoutant des garde-fous type spécification d'une page temporelle réduite, mais pour l'instant faisons simple.
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Valentin Deniaud a écrit :
Yep, oublié de mentionner ça dans la description, pour l'instant on a pas du tout envie de permettre de récupérer les données sans regroupement, ça risque de ne pas passer dans les tuyaux et l'affichage dans combo sera moche. Ça pourra être réactivé quand on en aura besoin, en ajoutant des garde-fous type spécification d'une page temporelle réduite, mais pour l'instant faisons simple.
Ok très bien, faisons comme ça.
Edit: Et dans doute aussi préciser, quand on documentera tout ça, la précédence du service sur l'ou, laquelle n'est plus évidente maintenant qu’on ne lève plus d'erreur de validation.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4a36966b557cdbef1337f487c20552bbb9a396a6
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Dec 2 16:13:57 2020 +0100
api: small adjustements to statistics endpoints (#49174)Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api: small adjustements to statistics endpoints (#49174)