Chat

[molro]

Corrección

Instrucciones Readme.md
Actualizadas

[StratocasterO]

Mauricio, he tenido dos errores y no puedo probarlo:

• El primero al arrancar la app de react. Me ha salido esto, pero he buscado y forzando el guardado del package.json se ha arreglado solo:
  

• El segundo no me deja registrar usuarios. La consola del explorador dice esto:
  

• El registro no hace devuelve ningun mensaje y no se sabe si se ha creado el usuario o no. Al hacer login la consola devuelve null, undefined y otras cosas así ❌

Date prisa a echarle un ojo y lo volvemos a mirar 😉

[StratocasterO]

Problemas que arreglar:

• La página se recarga al hacer login pero no me lleva al chat
• Aunque hago logout con un usuario sigue apareciendo en la lista de usuarios conectados
• La sala principal no tiene permanencia de mensajes (o añades permanencia o no permites escribir, una de dos)
• Me salen los mismos usuarios conectados en todas las salas

A parte de estos problemas funcionales, algunas buenas prácticas del back:

• Logs excesivos en el server, no hace falta mostrar por la consola cada intento de conexión, mensaje y cambio de sala
• Tienes trozos de código comentados por todas partes, limpia un poco esas cosas o haz una rama "limpia" y otra de trabajo

Dale un repaso, tienes que acabar esta semana sin falta! 🔥

[StratocasterO]

Buen trabajo, Mauricio, está mucho mejor! Algunas cosas a tener en cuenta:

• Bien hecho con la estructura del server y buen trabajo con TypeScript ✔
• Sobre el back, sigues teniendo muchos logs que deberían estar solo en dev, no en prod
• También habría que limpiar un poco algunas cosas: imports que no se usan, diversificar un poco más algunos archivos que son demasiado extensos, etc 🧹
• También veo muchos endpoints, parece que te has complicado más de la cuenta con eso (habría que hacer el máximo posible con sockets)

Sobre el front, que nos interesa menos en esta especialidad pero está bien que lo tengas en cuenta para añadir estas cosas al portfolio o así:

• Buen trabajo también haciendo el front en React, aun con la pinta ochentera 😜
• Es muy importante añadir verificaciones de email, contraseña y demás, ahora mismo acepta cualquier string como email
• También hay que comunicar al usuario de manera clara cualquier error: contraseña incorrecta, email no válido, login fallido...
• Ojo también con los logs en el front, también hay que limitarlos en prod

Acaba entre mañana y el finde las dos entregas de estructura de datos y el ejercicio de los dados para tenerlo corregido el lunes y hacer la defensa. Vamos, que ya lo tienes! 🔥