Feedback Manu #40
Description
Alberto felicitaciones por la presentación del proyecto. Está muy bien estructurado, lograste los objetivos de la entrega. Se nota mucho la evolución de tus entregas a la del día de hoy. La defensa estuvo muy bien hecha y demostraste aquellos aspectos relevantes no sólo a nivel de código y lógica si no también de mejoras y lecciones aprendidas. Me remito a las observaciones dadas en los últimos proyectos para que las tengas en cuenta siempre ya que responden a aspectos de buenas prácticas y uso de la tecnología. Durante tu presentación estuve tomando algunas observaciones que podrías tener en cuenta.
- Asegurate de hacer handdle errors. Esta es una buena práctica. Tienes muchos componentes y servicios que pueden romperse y no sabrás por qué o en qué momento del ciclo de vida del componente está fallando, especialmente porque ya hay anidación.
- Felicitaciones por quitar de las carpetas aquellos archivos que vienen por defecto y no se usan o no aportan.
- Hay estilos inline (alturas, anchos, transiciones…) por ahora es de aspectos muy concretos y gestionables, pero que no se nos convierta en una mala práctica.
- Tienes prácticas técnicas que son muy UI friendly, felicitaciones por hacer el overflow del body.
- Estás haciendo manipulación del dom, extrae este tipo de lógica en un servicio y encapsulalo y así no acoplas UI con Dom global.
- Los headers tienen hardcodeados los selectores (nombres y links), sería ideal llevarlos a un array u objeto dentro de una variable global, pues podría ser más escalable, reutilizable y optimizable con bucles o iteraciones.
- Felicitaciones por hacer componentes dinámicos
- Cuidado con los comentarios en los servicios y componentes. Si el código es descriptivo, no es necesario ponerlos
- Felicitaciones por el desarrollo de componentes escalables y reutilizables. Manejas muy bien las propiedades y las children properties.
- La gestión de errores y redireccionamientos, te recomiendo encapsularlos en servicios pertinentes, pues la lógica se mezcla con la responsabilidad del servicio donde lo creas (ejemplo: auth handdle error en github.service)
- Tienen funciones muy largas que hacen más de una cosa (obtener, filtrar, guardar) Revisa un poco más a profundidad el servicio de api de github
- La presentación fue la caña, muy inmersiva.
- Felicitaciones por el análisis funcional y técnico del proyecto, haces un mapa global del proceso
- Felicitaciones por hacer uso de tecnologías no propuestas durante el curso
- Felicitaciones por hacer un test e2e
- En el flujo de la presentación, hablaría de los test despues de mostrar el código y de las mejoras previtas justo despues de hacer la demo.
- Las transiciones le dan mucho nivel a la UI, felicitaciones.
- Resolver = Proxy
- tratar de no usar console.xxx por tema de seguridad y asincronías, en cambio, usa un sistema de logs que te ayuden a gestionar ese debuggin
- A nivel visual muy buen trabajo, hay mucha consistencia en todo visualmente