-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(authors): add authors api endpoints #23
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dejé algunas ideas respecto a cómo aislar las rutas de la API del resto de la aplicación.
¿Podrías además agregar en la descripción del PR un ejemplo del response body que obtienes en cada caso?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Si no es mucho problema, propongo un cambio que me tiró error en un test; no se valida un firstName
vacío, por lo que se guarda como null
en la base de datos y al esperar un post con status 400, me lo devuelve con 201 🙈 .
(Sino, hago el cambio en la PR de tests de endpoints).
@dasolari creo que eso debería ir en otro pull request que se encargue de agregar más validaciones al modelo de author. Puedo hacerlo en un rato |
Ah perfecto. Si quieres lo hago yo (en otro PR). Avísame @meretamal |
@dasolari dale tú entonces ❤️ |
What?
Se implementaron dos enpoints para manejar el recurso authors
Why?
Para poder proveer parte de la aplicación como una API.
El body del response en cada caso sigue el estándar jsonapi:
How?
Utilizando
koa-router
se creó un router con un métodoGET
y otroPOST
y se usójsonapi-serializer
para serializar el response de los requests.Anything Else? (optional)
Para probarlo es importante correr: