Feedback project 2 Data-dashboard

[AmaliaRiveraC]

1. Investigación, aún pueden llevar más lejos su curiosidad investigando sobre los métodos para manipulación de objetos como Object.values() o hasOwnerProperty(), que se refleje en su código si están investigando por su cuenta, yendo más allá de lo que se les da en el LMS o en las aulas.
2. JS: Semántica/Nomenclatura aguas con el nombre de generaciones que ponen en español, sin esta mezcal de lenguajes llegamos al nivel 4!
3. Funciones y arquitectura, cuidado en el archivo data.js están manipulando DOM, las responsabilidades de manipulación de la data y manejo de DOM que es vista deberíamos buscar separarlas!
4. Test y eslint: Incluyeron la guía de estilos, muy bien! Para el próximo proyecto les recomiendo correr el comando de npm run test y npm run eslint y ahí podrán ver y después corregir los warnings que les está lanzando según la propia guía de estilos que incluyen.
5. Estructura de datos: No han organizado la data ha manera de estado como solicitaban los detalles de implementación, es ideal comenzar a modelar los datos, agruparla en arrays u objects.
6. HTML correctitud/validación en index.html tienen 6 errores y 3 warnings aparte de correr htmlhint en consola validen la correctitud de su código antes de entregar en este link https://validator.w3.org/
7. Git: Les he dejado el nivel 3 de que hicieron Pull request y mergin pero les debo alertar que no están cumpliendo con el nivel 2 de git, que pide convenciones en la escritura de los commits, es decir que haya un estándar de escritura "los commits deben ser chicos y fácilmente abordables por un tercero", (ej. agrega funcion computeStudentsStats, agrega estilo de login, cambia insights de ux en readme) ustedes deberían llegar a un acuerdo al iniciar sus próximos proyectos para este tema. Las siguientes ocasiones si no se cumple con lo esperado en el nivel 2 no habrá forma de estar en el nivel 3.
8. Responsive: Veo que han puesto atención en el concepto de responsive y considero muy valioso ese esfuerzo aunque no es lo mismo un diseño adaptativo que se adapta a las distintas pantallas de diferentes dispositivos, sino que en responsive se piensa el diseño para cada una de estas pantallas, busquemos evitar los espacios en blanco debajo del gif que no aportan a la interfaz.

*Bien por las buenas prácticas! Por la documentación de la entrevista! ¡Las felicito por haber llegado hasta acá con tan buenos resultados, denle atención a sus áreas de oportunidad y crecerán rápidamente! Cualquier duda escríbanme.

[AmaliaRiveraC]

Por cierto, no ocuparon el README que se les proporcionó de guía en la carpeta de UX! También chequen cómo es la sintáxis para hipervinculos en markdown. Muchos saludos!

[Vicky2813]

Gracias Amalia estaremos muy pendientes de tus recomendaciones y consejos .. Apreciamos tu feedback

[DiLeyRa]

Gracias Amalia, tomaremos en cuenta el feedback para los siguientes proyectos y en cuanto puedamos mejoraremos los puntos a tratar.