Fabrica-de-Desarrollo-CE-FPUNE / bqs-service

2 stars 0 forks source link

Implementación de Rutas y Middlewares. #11

Closed DDelvRoj closed 3 months ago

DDelvRoj commented 3 months ago

Creación y implementación de estudianteRouter, contendrá los métodos de petición HTTP necesarios con relación a los estudiantes.

Se cumple con lo siguiente de momento:

  1. La api tiene que tener estas rutas
    • [x] get /api/estudiante/info
    • [x] get /api/dev/test
  2. otros:
    • [x] implementacion de middleware para el usuario
    • [x] Implementacion de rutas y controladores
elialm7 commented 3 months ago

espectacular, no te olvides de escribir tests para esas rutas :3

DDelvRoj commented 3 months ago

espectacular, no te olvides de escribir tests para esas rutas :3

Necesito ayuda sobre get /api/dev/test, si solamente debe ser para estudiante, o una nueva función en jest, hago lo segundo de todas maneras porque será necesario.

elialm7 commented 3 months ago

Con respecto al /api/dev/test, es basicamente una endpoint en donde vas a poder usar en tu frontend como test. es decir cuando hagas una request va a devolver un ejemplo del json que la endpoint real deberia devolver.

DDelvRoj commented 3 months ago

Con respecto al /api/dev/test, es basicamente una endpoint en donde vas a poder usar en tu frontend como test. es decir cuando hagas una request va a devolver un ejemplo del json que la endpoint real deberia devolver.

Debe ir incluida entonces en la carpeta routers verdad? @elialm7

elialm7 commented 3 months ago

asi es

DDelvRoj commented 3 months ago

Con estos últimos cambios ya se pudo realizar la separación de responsabilidades para los endpoints, implementé un errorHandler más robusto y hagan revisión de ErrorConStatusConstructor, este automáticamente crea el StackTrace que nos va a servir a nosotros, así como también envía la data al middleware para que funcione mejor, se puede implementar un sistema de logs por archivos o algo así, pero de momento creo que va bien.

DDelvRoj commented 3 months ago

Aún falta el middleware para poder manejar authentication token pero ese estaría mejor hacer en otro issue cuando tengamos definido una capa de servicio para usar en los controladores.

elialm7 commented 3 months ago

Me gusta, pero antes de dar mi ok, me gustaria que expliques un poquito que proposito, tiene la seccion de developer, en el sentido de que uso tendria en la implementacion del frontend, esa parte no logre entender su objetivo, despues el resto, expectacular, el unico cambio que te pediria es tener nombres mas consistentes con los archivos/metodos/clases y podemos mergear