Skip to content

Conversation

@molro
Copy link
Owner

@molro molro commented Mar 7, 2022

Entrega Niveles 1 al 3
Incluida colección de postman.

@StratocasterO
Copy link

Buen trabajo, Mauricio! Algunos comentarios:

  • Si pongo una ruta que no existe me devuelve el error predeterminado de Express
  • Si no pongo imagen, el server devuelve una string. Si no pongo nada, lo mismo ❌
  • La autentificación deberia ir en un middleware (con los valores correctos hardcodeados si quieres)
  • Ojo al hacer la colección de Postman, que tienes parametros para el login que no se usan (es importante no confundir al cliente)
  • Es un poco raro tener readme.ms en tantas carpetas. Esas notas mejor en comentarios
  • En el readme principal tendrian que estar reflejadas las rutas en forma de lista, para mejorar la documentacion

Bien hecho, solo queda pulir detalles 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants