API // Revisión de primer entrega

Michelle Torresrequested to merge
nmicht/api:master into development

En cada archivo podrán encontrar comentarios taggeados con FIXME para que puedan revisar los detalles de su proyecto y trabajar en las correciones.

Además de los detalles en la lógica y código del proyecto, deben trabajar en las siguientes correcciones:

  • Los metodos getAll deben manejar paginado y filtrados
  • Se les pidió que manejaran clases y los controladores los dejaron como funciones sueltas
  • Recomiendo usar async/await para el manejo de los promises en lugar de los then y catch que tienen
  • Falta una abstracción para estandarizar los errores y no estar repitiendo lo mismo por todo el código
- deberia validar que sea un asiento valido, no nada mas que no este vacio __FIXME__ [src/middlewares/validTicketBody.js](src/middlewares/validTicketBody.js)
- deberia validar que sea una siento valido, no nada mas que no este vacio __FIXME__ [src/middlewares/validSeatBody.js](src/middlewares/validSeatBody.js)
- deberia validar que sean datos valido, no nada mas que no este vacio __FIXME__ [src/middlewares/validTicketDetailBody.js](src/middlewares/validTicketDetailBody.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/seats.js](src/models/seats.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/directors.js](src/models/directors.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/emails.js](src/models/emails.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/functions.js](src/models/functions.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/movies.js](src/models/movies.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/rooms.js](src/models/rooms.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/actors.js](src/models/actors.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/tickets.js](src/models/tickets.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/ticketsDetail.js](src/models/ticketsDetail.js)
- El modelo no debe ejectuar queries ni diseñarlos, estos metodos deben existir en la clase abstracta de manejo de base... __FIXME__ [src/models/users.js](src/models/users.js)
- esta debia ser una clase abstracta para realizar todas las diferentes manipulaciones de la base de datos __FIXME__ [src/models/connector.js](src/models/connector.js)
- Falta validar cuerpo del request y parametros __FIXME__ [src/routes/movies.js](src/routes/movies.js)
- la validacion deberia ser de rooms y no de user __FIXME__ [src/routes/rooms.js](src/routes/rooms.js)
- la validacion deberia ser de rooms y no de user __FIXME__ [src/routes/rooms.js](src/routes/rooms.js)
- mensajes de log no deben llegar a master NUNCA __FIXME__ [src/models/users.js](src/models/users.js)
- se debe validar el cuerpo del request __FIXME__ [src/routes/TicketDetail.js](src/routes/TicketDetail.js)
- se debe validar el cuerpo del request y los parametros __FIXME__ [src/routes/tickets.js](src/routes/tickets.js)
- se debe validar el parametro __FIXME__ [src/routes/actors.js](src/routes/actors.js)
- se debe validar el parametro __FIXME__ [src/routes/actors.js](src/routes/actors.js)
- se debe validar el parametro __FIXME__ [src/routes/actors.js](src/routes/actors.js)
- se debe validar el parametro __FIXME__ [src/routes/actors.js](src/routes/actors.js)
- se debe validar el parametro __FIXME__ [src/routes/TicketDetail.js](src/routes/TicketDetail.js)
- se debe validar el parametro __FIXME__ [src/routes/directors.js](src/routes/directors.js)
- se debe validar el parametro __FIXME__ [src/routes/directors.js](src/routes/directors.js)
- se debe validar el parametro __FIXME__ [src/routes/directors.js](src/routes/directors.js)
- se debe validar el parametro __FIXME__ [src/routes/directors.js](src/routes/directors.js)
- se debe validar el parametro __FIXME__ [src/routes/functions.js](src/routes/functions.js)
- se debe validar el parametro __FIXME__ [src/routes/functions.js](src/routes/functions.js)
- se debe validar el parametro __FIXME__ [src/routes/functions.js](src/routes/functions.js)
- se debe validar el parametro __FIXME__ [src/routes/functions.js](src/routes/functions.js)
- se debe validar el parametro __FIXME__ [src/routes/TicketDetail.js](src/routes/TicketDetail.js)
- se debe validar el parametro __FIXME__ [src/routes/rooms.js](src/routes/rooms.js)
- se debe validar el parametro __FIXME__ [src/routes/users.js](src/routes/users.js)
- se debe validar el parametro __FIXME__ [src/routes/rooms.js](src/routes/rooms.js)
- se debe validar el parametro __FIXME__ [src/routes/TicketDetail.js](src/routes/TicketDetail.js)
- se debe validar el parametro __FIXME__ [src/routes/rooms.js](src/routes/rooms.js)
- se debe validar el parametro __FIXME__ [src/routes/rooms.js](src/routes/rooms.js)
- se debe validar el parametro __FIXME__ [src/routes/seats.js](src/routes/seats.js)
- se debe validar el parametro __FIXME__ [src/routes/seats.js](src/routes/seats.js)
- se debe validar el parametro __FIXME__ [src/routes/seats.js](src/routes/seats.js)
- se debe validar el parametro __FIXME__ [src/routes/seats.js](src/routes/seats.js)
- se debe validar el parametro __FIXME__ [src/routes/TicketDetail.js](src/routes/TicketDetail.js)
- se debe validar el parametro __FIXME__ [src/routes/users.js](src/routes/users.js)
- se debe validar el parametro __FIXME__ [src/routes/users.js](src/routes/users.js)
- se debe validar el parametro __FIXME__ [src/routes/users.js](src/routes/users.js)
- se debe validar tmb el formato, no solo vacio __FIXME__ [src/middlewares/validFunctionBody.js](src/middlewares/validFunctionBody.js)

No encontre la coleccion de postman, y dado que no hicieron la entrega presencial, no hay manera de evaluarles completo.

Merge request reports