Utiliser une instance d'Error plutôt qu'un objet pour signaler une erreur avec OpenFisca

betagouv / mes-aides-api

A mapping interface between the mes-aides user interface and the OpenFisca simulation engine.

https://mes-aides.gouv.fr

20 stars 2 forks source link

Closed guillett closed 7 years ago

guillett commented 7 years ago

Cela facilite le développement en ajoutant la pile d’exécution au niveau de l'API (coté Express serveur).

L'idée est de remplacer ça: capture d ecran 2017-01-29 a 16 55 02

En: capture d ecran 2017-01-29 a 16 50 25

J'ai vu que https://github.com/sgmap/mes-aides-ui/blob/master/app/js/controllers/resultat.js#L73 fait référence à error.apiError mais je ne crois pas que ce soit fonctionnel en l'état (En effet, dans mes tests la valeur renvoyée était '[object Object]' et non pas un objet en JSON).

MattiSG commented 7 years ago

Merci de documenter les raisons qui mènent à fermer une PR ou une issue :wink: