[Michael Vega]

Hola Diego, el CRUD de mascotas es para ambos roles. Las citas es casi igual, la excepcion es actualizarlas y eliminarlas. Además, un usuario puede ver las citas y mascotas propias, pero el admin el de todos. Igualmente ya no recibo mas actualizaciones. Yo califique el fin de semana lo último que habia.

• This reply was modified hace 1 month, 1 semana by  Michael Vega.

[Michael Vega]

Si tiene problemas con la carga del frontend, por favor, cree un repositorio aparte y lo comparte.

[Michael Vega]

Se revisa nuevamente el proyecto, a nivel de frontend, siendo usuario no me lista mis mascotas, pero siendo admin si las muestra. En el modulo de citas, no me lista mis mascotas siendo usuario, pero si las lista siendo admin. Deberia listarse para ambos roles, pero solo quien puede eliminar una cita es el admin.

Se actualiza puntaje 80/100

[Michael Vega]

Revisando esta muy bien el README donde da todas las indicaciones necesarias para poner en funcionamiento las aplicaciones (backend y frontend). Se revisa todo el flujo, aunque existe un error de validacion con respecto al campo raza y edad para las mascotas, se muestra como opcionales, pero si no las envio genera un error y no redirige nuevamente al form.

Puntaje actualizado: 70/80

[Michael Vega]

Revisando el proyecto, veo que solo se puede crear mascotas y citas siendo usuario, pero un administrador tambien debe poder crear. Siendo administrador, puedo ver todos los datos, modificar citas si es necesario y eliminarlas. Existe un error en la carga de la fecha cuando se edita una cita, pues no carga. En mascotas no puedo realizar ningun proceso solamente puedo ver el listado.

Se recomienda hacer el crud para todos, pero manteniendo las recomendaciones. Viendo que tiene los servicios backend funcionales.

Puntaje actualizado: 70/80

[Michael Vega]

Se revisa el codigo, carga el frontend de citas, pero tiene algunos errores en cuanto a los roles. Tiene mal nombrado el permiso que necesita para el proceso de creacion. Se recomienda que el campo de ID de la mascosta sea reemplazado por un combo con los nombres de mi mascota y asi pueda elegirlo, muchas veces no nos acordamos de los ids.

Puntaje actualizado: 77/80

[Michael Vega]

Existe un error en el manejo del token, pues cuando se vence no me redirige al login. Deberia eliminar toda la data que se guarda en el localStorage y redirigir.

Persiste el error en la creacion de citas por el doctorId, pero no veo a nivel de front donde se crean los doctores. Error:

insert into appointments (date, doctorId, id, notes, petId) values (‘2025-05-27’, 0, 0, ‘Prueba 1’, ‘5’)”
​​
sqlMessage: “Cannot add or update a child row: a foreign key constraint fails (pet_app.appointments, CONSTRAINT appointments_doctorid_foreign FOREIGN KEY (doctorId) REFERENCES doctors (id) ON DELETE CASCADE)

Funciona ya la carga de creacion de mascotas.

Puntaje actualizado: 75/100

[Michael Vega]

El comando de las migraciones no se encuentra, se recomienda algo como

"make:migrations": "knex migrate:latest"

Se pueden crear las mascotas, pero la data no carga una vez creada, se debe hacer refresh.
No puedo crear citas, genera un error 500.

Puntaje actualizado: 70/100

[Michael Vega]

Se revisa nuevamente el proyecto, funciona el login, registro, crud mascota, crud de citas. Falta manejo de roles en las acciones de completar una cita y el de listar todas las mascotas y citas, cuando el usuario es un Admin.

Puntaje actualizado: 80/100

[Michael Vega]

Desde el Front no puedo crear mascotas ni citas. Falta la documentacion para saber que debe agregarse en el .env, se puede crear un .env.example con las variables sin datos, solamente la estructura.

Puntaje actualizado: 55/100

• This reply was modified hace 1 month, 1 semana by  Michael Vega.

[Michael Vega]

Hola Maria, que pena el dia de ayer tuve un inconveniente que no me permitio poder responder y pues tampoco conectarme. Nos reunimos el dia de hoy.

[Michael Vega]

Si, me confirma cuando tenga las correcciones para poder verificar

[Michael Vega]

No se encuentra el archivo de configuracion o seed, para poder ejecutar las migraciones y que se creen las tablas usando el ORM. No es posible revisar funcionamiento. Se revisa estructura bien implementada. Se revisa el desarrollo front, igualmente bien implementado.

Por favor,. realizar los ajustes respectivos. Puntaje actual: 50/100

• This reply was modified hace 1 month, 2 semanas by  Michael Vega.

[Michael Vega]

Se revisa el proyecto, buena estructura de rutas, faltan algunas validaciones, falta la seccion de citas. Maneja bien el control de obtener datos del usuario usando token y cargando los datos segun el usuario en sesion. Buen manejo de roles y permisos.

Puntaje actual: 75/100

[Michael Vega]

Se revisa nuevamente el proyecto. Puntaje actualizado: 70/100.