search

openpolitica / open-politica-backend

Backend del Proyecto Open Política construido en Node.js
Apache License 2.0
0 stars 0 forks source link

Added error handler #91

Closed dmarquinah closed 3 years ago

dmarquinah commented 3 years ago

Added error handler to change the statusCode of the request. Try /policies/questions?

gianpietro1 commented 3 years ago

@dmarquinah esta acción está bloqueada en frontend (el botón no se activa hasta que no selecciones tres), ¿sería igual necesario aquí?

dmarquinah commented 3 years ago

Si, esto debido a que luego probablemente se usará algún módulo de error en frontend y quieran mostrar algún mensaje. De todas maneras siempre es bueno tener esos mensajes de error para mapear los casos en los que no debería de funcionar el API.

gianpietro1 commented 3 years ago

@dmarquinah no está funcionando con un solo topic desde swagger, porque para un solo tópico el llamado se hace así, y es un string. /policies/questions?topics=governability ¿Crees que haya que arreglar desde swagger para que siempre sea un array? (habría que ver cómo en swagger) y confiar en que front siempre hará el llamado formando un array? o debemos lidiar con este caso en el API?

dmarquinah commented 3 years ago

@gianpietro1 tal cual, frontend hace un encode del array en url, parece que swagger no lo hace así. Voy a investigar sobre ese cambio en swagger, pero creo que esto igual ya se podría ir haciendo merge. Además, se acordó con frontend el usar ese param como array así que no creo que luego vayan a cambiarlo o al menos no veo razón para hacerlo por lo que no tendríamos problemas.