Development #33863
connecteur pass urbain lille
0%
Description
Une première phase avec juste un appel pour demander la carte (/le pass).
Fichiers
Révisions associées
lille urban card: consider more boolean attributes (#33863)
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-add-basic-connector-lille-urban-card-33863.patch 0001-add-basic-connector-lille-urban-card-33863.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
C'est un service adhoc derrière, plutôt peu carré, mon idée initiale était de juste accepter toutes les données du formulaire wcs tel qu'il existe et convertir ça mais il y a des manquements dans la demande, et des trucs pas top de toute façon, donc voilà, non.
Mis à jour par Thomas Noël il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Même pas envie de tester un peu la réponse des self.requests.post(...) avant de faire le « .json() » dans la foulée ? Genre, si c'est pas 200, ou si la réponse n'est pas du json, ie quand tout est planté derrière... Bon, c'est vrai que si ça plante on aura le check_status qui aura planté aussi et ne nous polluera pas d'alertes. Laissons tomber pour l'instant, on verra à l'usage si les pannes sont peu fréquentes ça nous fera un système d'alerte efficace, sinon on améliorera la gestion des erreurs.
Sinon classiquement on ferait un return {'data': response}
mais comme y'a quasiment rien dans la réponse, je laisse filer aussi.
Ack donc, j'aurais ici juste montré que j'ai relu en vrai.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
J'ai tapé la réponse dans une clé "data".
commit 06a05d5a9bebe93494200d8ca685f8937b038487 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Jun 12 10:02:07 2019 +0200 add basic connector for lille urban card (#33863)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add basic connector for lille urban card (#33863)