Sprint42

[molro]

Entrega Nivell 1

[StratocasterO]

No puedo probarlo, Mauricio! Algunas cosas que veo:

• No me dices donde tengo que cambiar la info del user de MySQL (para eso está el .env)
• El archivo .env no se sube al repo. Se puede subir un .env-template para indicar al cliente lo que tiene que cambiar
• Cuando lo encuentro y lo cambio, peta (me dice que unknown database)
• Lo mismo que te dije del anterior: ojo con tener un readme en cada carpeta, es muy confuso. Esas notas en comentarios o en el readme general o en otro archivo con todas las notas, pero no ahi!
• La estructura tiene buena pinta

Dale un repaso a esas cosas, que son más de forma que de contenido, a ver si puedo arrancarlo! 😉

[StratocasterO]

Hay cosas que corregir para darlo por bueno, Mauricio!

• Con el POST player y el UPDATE playerpuedo crear jugadores con nombres repetidos y sin nombre (poniendo '' en el campo). No indicas que para los anónimos tengo que dejar el body en blanco
• El listado de jugadores devuelve un array, no un json
• El GET de las jugadas de un jugador devuelve el jugador, no sus jugadas
• Hay endpoints que no tienen el nombre que se requiere (p.e. lastplayer, firstplayer)
• No veo el endpoint DELETE para borrar las jugadas de un jugador
• Las rutas extras devuelven el HTML de Express, no un json con 404

Dale un repaso a estas cosas que me lo pueda mirar mañana a primera hora!

[molro]

Resueltos los puntos.

[StratocasterO]

Buen trabajo corrigiendo errores, Mauricio. Algunas cosas para acabarlo de pulir:

• El server hay que arrancarlo dos veces para que cree la base de datos y luego cree las tablas, habría que solucionarlo
• La ruta PUT dice que "jugador modificado" pero no modifica ❓
• Lo de escribir Invitado es muy raro, tendria que crearlo anónimo cuando se deja en blanco
• Cuando no hay contenido no debería devolver 200 con el JSON vacío, debería devolver un mensaje de que no hay contenido (tiradas, jugadores o lo que sea) ❌
• Muchos logs, ya lo sabes, hay que limpiar eso ❗
• Bien hecho con la estructura ✔
• Bien hecho añadiendo los 404 ✔

Arregla lo de la base de datos que te falta y estás listo para proyecto (pero ten en cuenta todos estos comentarios cuando estés allí 😉).