Bug #13295
MandayeJS : GET sur l'api doit renvoyer les locators sous une key @locators@
0%
Description
L'API renvoit ,en GET, directement les locators de l'Application. Pratique pas du tout recommandée vu que ceux-ci varient en fonction de l'application.
Faudrait les renvoyer sous la forme :
{
"locators": [
]
}
Fichiers
Demandes liées
Historique
Mis à jour par Josué Kouka il y a plus de 7 ans
- Fichier 0001-api-return-locators-under-key-locators-13295.patch 0001-api-return-locators-under-key-locators-13295.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 7 ans
Je ne sais pas le but du truc, mais clairement ça ne fait pas que mettre la réponse sous une clé "locators" : ça en change aussi le format. Donc, au mieux, le message de commit et le titre du ticket sont pas top.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Et comme le rappelait Benjamin, passons un peu plus de temps sur la description des tickets (quoi faire ? pourquoi ? pour qui ? etc.)
Mis à jour par Josué Kouka il y a plus de 7 ans
Frédéric Péters a écrit :
Et comme le rappelait Benjamin, passons un peu plus de temps sur la description des tickets (quoi faire ? pourquoi ? pour qui ? etc.)
Si je me rappelle bien des discussions d'il y'a 6 mois, l'idée était de permetter la gestion des liaisons depuis d'autres entitées Publik, principalement Combo.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Entre ça et le titre actuel, il y a un plan qui manque.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Je ne vois toujours pas; on pourrait en longueur détailler le truc qu'on voudrait pouvoir faire, pourquoi ça foire aujourd'hui, pourquoi il faudrait faire ceci ?
Mis à jour par Josué Kouka il y a plus de 7 ans
Frédéric Péters a écrit :
Je ne vois toujours pas; on pourrait en longueur détailler le truc qu'on voudrait pouvoir faire, pourquoi ça foire aujourd'hui, pourquoi il faudrait faire ceci ?
Rien ne foire, l'API marche correctement. Je veux juste changer le format de la valeur retour.
Mis à jour par Frédéric Péters il y a plus de 7 ans
J'ai bien compris le patch mais pas encore sa raison d'être.
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
Je suis contre le fait de renvoyer des dictionnaires JSON dont les clés sont variables; ce n'est juste pas le style qu'il faut pour nos APIs. Aussi renvoyer la donnée simple comme un dico empêche d'étendre ce endpoint. Donc plutot que
{ name => X, .. } on renvoie { 'locators' => [ { 'name' => name, X }, .. ].
On peut changer cette API parce qu'elle n'est utilisée nulle part.
Mis à jour par Frédéric Péters il y a plus de 7 ans
On peut changer cette API parce qu'elle n'est utilisée nulle part.
Elle est arrivée via #9782 mais usage inconnu. Si ce n'est aujourd'hui toujours pas utilisé, je serais pour qu'on ne développe pas et qu'on s'interroge d'abord sur l'objectif… Parce que s'il s'agit comme écrit plus haut de permettre la gestion des liaisons depuis combo, il faut se coordonner avec ce qui doit se faire côté Passerelle, et les plans présents (ticket?) ne correspondent du tout à cette API.
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
- Priorité changé de Normal à Bas
Effectivement je ne trouve pas de ticket/roadmap écrit sur le sujet d'une cellule liaison dans combo ou ailleurs, je vais baisser la priorité de ce ticket jusqu'à ce qu'on s'accorde sur quelque chose.
Mis à jour par Thomas Noël il y a plus de 7 ans
J'ai hier un peu fini par comprendre à quoi servait cette API, le nom "locators" étant particulièrement mal choisi je trouve, qui m'avait enduit d'erreur ;)
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Lié à Development #13327: cellule de gestion des liaisons ajouté