Development #21011
Rendre APIError importable depuis passerelle.utils.api
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
05 janvier 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
On se trouve généralement à avoir :
from passerelle.utils.api import endpoint from passerelle.utils.jsonresponse import APIError
Ça me semblerait plutôt pas mal de pouvoir avoir l'APIError importé depuis la première ligne.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-misc-make-APIError-importable-from-passerelle.utils..patch 0001-misc-make-APIError-importable-from-passerelle.utils..patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Juste la mise à disposition, je ne passe pas sur tous les connecteurs.
Mis à jour par Thomas Noël il y a plus de 6 ans
Avec un commentaire qui rappelle de ne pas retirer cet import, qu'il est là pour aider aux imports. (un # noqa -- imported here to improve Frederic's life) (au moins le # noqa pour ne pas faire crier mon vim ?)
(Ou alors déplacer la déclaration dans api.py, mais c'est plus simple qu'elle reste dans jsonresponse où elle est utilisée)
Bref, ack, avec ou sans # noqa
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
J'ai ajouté un commentaire; mais pas de "# noqa" sur l'idée qu'une fois réellement utilisé (via #20705) il n'y aura plus d'alerte.
commit 81696df4d8e5e1b287b2bf9848b1c0a162705188
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Jan 5 14:15:17 2018 +0100
misc: make APIError importable from passerelle.utils.api (#21011)
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
misc: make APIError importable from passerelle.utils.api (#21011)