Développement #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.
Files
Associated revisions
History
Updated by Valentin Deniaud almost 4 years ago
- File 0001-api-small-adjustements-to-statistics-endpoints-49174.patch 0001-api-small-adjustements-to-statistics-endpoints-49174.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Paul Marillonnet almost 4 years ago
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 ?
Updated by Valentin Deniaud almost 4 years ago
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.
Updated by Paul Marillonnet almost 4 years ago
- Status changed from Solution proposée to 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.
Updated by Valentin Deniaud almost 4 years ago
- Status changed from Solution validée to 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)Updated by Frédéric Péters almost 4 years ago
- Status changed from Résolu (à déployer) to Solution déployée
api: small adjustements to statistics endpoints (#49174)