mirenovi / cdmx-2018-06-bc-core-am-data-dashboard Goto Github PK

Hola @MirEnovi y Compañera
He estado revisando el proyecto y tengo algunos comentarios:

1. @MirEnovi debes agregar como colaboradora a AuraMelisa01, es extremadamente necesario pues los issues van guiados hacia ambas.

2. De la mano del punto anterior veo que no realizaron los issues y milestones correspondientes al sprint 2 del proyecto, que pasó con eso?

3. Que pasó con el deploy a Github Pages? era uno de los requisitos para el demo del sprint 1

4. Felicitaciones por el uso de diagrama de flujo del proyecto y de pseudocódigo, les servirá muchisimo a no solo entender el como se debe ver y como se interactua, sino que también a entender de donde viene la información y como deben internamente conversar con la data para obtener lo que necesitan.

5. Ya veo un trabajo lógico pero les recomiendo comenzar a trabajar con la data a profundidad lo antes posible (se encontrarán con toda la lógica para jalar la información y hacer los cálculos) y plasmar tanto visual como funcionalmente lo que tienen pensado para el proyecto.

6. Recuerden utilizar HTML semántico <header></header> <section></section> <aside></aside> <footer></footer> <article></article>

7. Ojo con la indentación en el archivo index.html un código bien armado, ordenado y limpio es esencial para quien hace codereview, da feedback o colabora con el proyecto. Les dejo esto

Vamos comiencen luego con a darle forma al datadashboard...you can do it! 💪

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.
2. JS Nomenclatura/Semántica: En general en nombramiento de variables y de funciones resulta descriptivo y utilizan camelCase aunque en main.js hay nombres en inglés y también en español y algunos nombres como clickEstudiantesTodos, containerG son inconsistentes, cuidemos estos pequeños detalles!
3. HTML correctitud validación en index.html tienen 3 errores y 1 warning y en index1.html tienen 2 errores y un warning, aparte de correr htmlhint en consola validen la correctitud de su código antes de entregar en este link https://validator.w3.org/
4. 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 el scroll horizontal que ahorita esta presente en el producto y los espacios en blanco.
5. 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.
6. Github espera el uso de issues y milestones, no llegan al nivel esperado, hubo un taller con Alexandra para este tema de los issues en github , serie de issues como guía para el desarrollo de tareas y/o de problemas y creación y uso de milestones. Pongan atención en este punto de la rúbrica para llegar al nivel, no tiene mucha dificultad y les ayudará a su planificación.

UX:

1. Entrevistas: Les recomiendo comenzar a documentar también el proceso de entrevistas, no sólo las preguntas que comentan haber hecho sino la entrevista misma, esto para tener material suficiente para evaluar de manera más objetiva cómo se están dando estas. Esta documentación también permitirá tener más material para entender el nivel de User centricity.
2. Contraste: Creo que existe cierto contraste entre el logo y los select que están a un lado, pero en la pantalla que nos muestra el detalle de los porcentajes por sedes y generaciones no veo contraste dentro de las tarjetas, y lo mismo pasa en la lista de estudiantes, no hay contraste en los datos que se muestran de las estudiantes y las negritas que le dieron a todos esos textos están en el mismo nivel del negro del logo. Cuidado! Busquemos contraste en los detalles más sutiles.
3. Tipografía: han cambiado la fuente por defecto y eso es bueno, ser curiosas, indagar en las distintas posibilidades, aunque también es recomendable apegarnos, según los resultados de nuestra investigación a la identidad visual de cada marca para llegar sin lugar a dudas al nivel 2.

*Buenas prácticas: recuerden que mientras que en js la buena práctica es nombrar a las variables y funciones con camelCase, los nombres de id´s, clases y carpetas como buena práctica se escriben con guión medio (ej. #card-color, login-form, style-session.css). Por otro lado y en el mismo tema de las buenas prácticas cuidado con indentación en archivos css, index_1.html y main.js

Sigan trabajando así de duro y busquen estrategias para atacar esas áreas de oportunidad. Llegarán lejos si así lo hacen. Cualquier duda escríbanme.