Development #51012
Solis AFI MSS : ajouter deux endpoints pour selectionner les enfants selon leur âge
Début:
10 février 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Pour éviter de faire du python dans wcs, lors de la définition des sources de données.
Fichiers
Historique
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a environ 3 ans
- Fichier 0001-solis_afi_mss-add-2-new-children-endpoints-51012.patch 0001-solis_afi_mss-add-2-new-children-endpoints-51012.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 3 ans
Deux détails :
- en cas de pépin lors du parse_date(now_date_string), les APIError doivent renvoyer un code http 400 (requête invalide).
- je ne comprends pas trop les timezone sur du calcul d'age (il s'agit juste de faire la différence entre deux dates, y'a pas de notion de timezone à gérer a priori)... y'a une raison à cette gymnastique ?
Mais surtout, tu as mis la même description sur les deux endpoints, faut changer... et en fait mieux : ça fait plutôt penser à mettre ces filtrages sur l'âge dans le endpoint children, en y ajoutant des paramètres optionnels "max_age" et "min_age" et "now", et les prendre en compte quand ils sont présents (ce qui permettra aussi d'afficher un jour "les enfants entre 6 et 11 ans")
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a environ 3 ans
- Fichier 0001-solis_afi_mss-add-age-selections-on-children-endpoin.patch 0001-solis_afi_mss-add-age-selections-on-children-endpoin.patch ajouté
Remarques prises en compte, merci.
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Noël il y a environ 3 ans
Vu en parallèle : patch qui pourrait être annulé car le besoin fonctionnel n'existe peut-être pas.
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus de 2 ans
- Statut changé de Solution validée à Rejeté
(demande anticipée et qui ne se révèle pas nécessaire)