Development #70348
Crash SMS Factor sur l'update des crédits restants
0%
Description
curl -H "Accept: application/json" -H "Authorization: Bearer $TOKEN" 'https://api.smsfactor.com/credits'
nous retourne un joli : {"status":1,"credits":"97","postpaid":"0","message":"OK"}
Notez que credits
n'est pas un nombre mais une chaîne de caractère. Je m'attendais à ce que ça soit un entier, la doc montre le contraire : https://dev.smsfactor.com/en/api/sms/account/credits
Fichiers
Révisions associées
Historique
Mis à jour par A. Berriot il y a plus d'un an
(notez également que sur https://dev.smsfactor.com/en/api/sms/send/send-single, on récupère bien les crédits sous forme d'entier…)
Mis à jour par A. Berriot il y a plus d'un an
- Fichier 0001-smsfactor-handle-credits-typed-as-string-70348.patch 0001-smsfactor-handle-credits-typed-as-string-70348.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par A. Berriot il y a plus d'un an
Mis à jour par A. Berriot il y a plus d'un an
Valentin Deniaud a écrit :
Tu peux mettre à jour les tests ?
J'avais oublié que j'en avais écrit tu vois :')
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Nickel (sous réserve de jenkins vert tout ça)
Mis à jour par A. Berriot il y a plus d'un an
Jenkins rouge mais je me permet de merger : c'est jenkins qui a été sous l'eau, les tests sont bien passé et l'étape de pylint qui a crashé passe en local.
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 083fab1f52d8b675d4422a66c65a241e24673ec3 Author: Agate Berriot <aberriot@entrouvert.com> Date: Mon Oct 17 15:30:22 2022 +0200 smsfactor: handle credits typed as string (#70348)
Mis à jour par Frédéric Péters il y a plus d'un an
Il est utile de ne pas supprimer (immédiatement) les branches wip, pour pouvoir retrouver le job jenkins.
Ici sur main ça échoue : https://jenkins.entrouvert.org/job/passerelle/3177 , les tests sont marqués ok parce qu'ils marchent dans un environnement tox mais pas un autre (py3-django22-xmlschema10).
Mis à jour par A. Berriot il y a plus d'un an
Frédéric Péters a écrit :
Il est utile de ne pas supprimer (immédiatement) les branches wip, pour pouvoir retrouver le job jenkins.
Ici sur main ça échoue : https://jenkins.entrouvert.org/job/passerelle/3177 , les tests sont marqués ok parce qu'ils marchent dans un environnement tox mais pas un autre (py3-django22-xmlschema10).
J'avais vérifié, ça passé sur tous les env tox jenkins sauf le pylint. L'echec sur https://jenkins.entrouvert.org/job/passerelle/3177/consoleFull est du à un problème de charge sur le noeud Jenkins pour autant que je puisse en juger (j'ai eu le même de manière intermittente sur ma branche aujourd'hui).
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
smsfactor: handle credits typed as string (#70348)