TeoVogel
commented
1 year ago ✅ Se puede registrar un usuario desde /register ✅ No hacen uso de los templates de blade para las vistas
⚠️ No hacen uso de las herramientas de más alto nivel de Eloquent para las queries (pero ver camisetasPorLiga en CamisetaApiController, es necesario ese join?)
❌ Se pueden crear camisetas con precio negativo, error en la validacion (solo hicieron validación en el html, pero en el controller no se hace validación alguna, por lo que se pueden seguir creando entidades con atributos incorrectos)
❌ Swagger no está funcionando (esto ahora se puede testear pero crear un pedido con un id de cliente o de camiseta que no existe retorna un error 500 y muestra todo el mensaje de error)
"❌" es un cambio requerido, el cual consume un crédito "⚠️" es una observación de posible mejora, no se requiere un cambio, no consume créditos
Errores encontrados: ❌ Se puede registrar un usuario desde /register ❌ Se pueden crear camisetas con precio negativo, error en la validacion ❌ No hacen uso de los templates de blade para las vistas ❌ No hacen uso de las herramientas de más alto nivel de Eloquent para las queries ❌ Swagger no está funcionando ⚠️ Cuidado con el contraste de los textos. En algunas pantallas hay texto blanco sobre un fondo claro
Sobre los créditos: El domingo 14 de Mayo vencieron 6 créditos. Como la entrega (el PR) fue hecho antes de esa fecha, los cambios requeridos consumieron sobre esos 6 créditos. Si no hay más de 6 "❌", significa que aún tienen la cantidad máxima posible de créditos hasta el momento.
Cualquier duda la charlamos el Martes en clase o por discord, manteniendo la consulta breve y ágil.