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

Documentación: Bien por agregar instrucciones de instalación, sería bueno que fueran más específicas con respecto a instalar npm y correr el comando de npm install. También se agregaron algunos comentarios. Llegamos al nivel esperado en este punto.
ESLINT: Bien por incluir la configuración para la guía de estilos en el archivo .eslintrc 👍
Git: mejora en el estándar de escritura de commits, llegamos al nivel 2, para el 3 se necesitaban los pull request.
UX:
Entrevistas: sube al nivel esperado!
Alineación: Se agregaron cards, lo cual mejora la alineación, cuida la jerarquía de la data dentro de tus cards, ¿todos los datos son igual de importantes?

Buenas prácticas: recuerda la ´key word´ para nombramiento de variables/funciones, const o let.
El proyecto ya cumple con el criterio de aceptación. Felicidades por el esfuerzo Rocío, una disculpa por la demora y por el detalle del slack. Sigue esforzándote tanto como hasta ahora y llegarás lejos. Un abrazo!

1. Documentación: Usar comentarios útiles en archivos de código js, css, html, textos que nos permitan entender métodos, propiedades, estructura en html por ejemplo! Al readme le hace falta la información sobre instalación y uso.
2. Eslint (estilo): Alexandra dio un pequeño taller donde les habló de uso de guía de estilos en js, configurando el archivo .eslintrc, ella les presentó la guía de estilos que usa Laboratoria, pongan atención en este tema para subir su nivel!
3. Funciones/modularidad: deberíamos buscar que la manipulación de la data (todas esas línea de código donde accedemos a propiedades específicas de nuestro json y donde las almacenamos en variables, o modelamos en estructuras de datos complejas, quede bajo la responsabilidad exclusiva de data.js. Este mismo punto afecta el nivel alcanzado en:
4. Arquitectura, donde aún no estamos separando responsabilidades o aplicando algún patrón de diseño.
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. Test: en este siguiente proyecto pongamos atención en el área del testing, tratemos de modificar la manera de desarrollar y orientar nuestro desarrollo a las pruebas que queremos pasar!
7. HTML: Semántica, utilizar etiquetas semánticas como main, article, section, aside, footer y desde el uso pensado de estas, ir expresando la arquitectura de la información!
8. Git: 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. Por otro lado no veo ninguna contribución en el flujo de git de Edith, con lo cual no alcanzan el requisito de pull request.
   UX:
9. Entrevistas: Veo que en el README de ux mencionan que utilizaron entrevistas semi estructuradas, pero no han documentado estos procesos y tampoco veo una mención explicita a estas, si me equivoco díganme!

• Apliquemos el feedback obtenido, pongamos atención en todo lo relacionado con JS y HTML, en la medida que vayamos mejorando estas áreas de oportunidad también nuestros alcances serán mucho mayores. Cualquier duda o aclaración escríbanme!,

Aplicar el Feedback al Sketch
Modificar Diagrama de Flujo
Json, Fetch, XHR

Hola @rxmstrd @DianaAr8 y @EdithGarciaDieguez
He estado revisando el proyecto y tengo algunos comentarios:

1. Ojo en el readme que tienen dentro de src/ux tienen o tuvieron conflictos en el archivo, pues no borraron el código que les deja git para indicar donde están los problemas, son así: <<<<<<< HEAD los tienen al menos 5 veces repetido, ojo con esos detalles.

2. Buen trabajo de investigación del usuario, se nota la dedicación e interés en saber las necesidades exactas del usuario, espero puedan lograr llevar a cabo el prototipo que plantean lo más fiel posible.

3. En el archivo index.html también tienen marcas de conflictos no resueltos en la línea 3, 46 y 56.

4. Por qué tienen 4 archivos js? dos en src sueltos y dos más dentro de js que a su vez está dentro de src. Deben cuidar esos detalles, está bien ordenar los archivos como estimen conveniente, pero no duplicar.

5. Cuando el test dice debería exponer función computeStudentsStats en objeto global quiere decir que computeStudentsStats es una función, no un método de un objeto datasheet como lo están haciendo en el proyecto, son funciones comúnes y corrientes, así es sucesivamente con todas las otras funciones. (window.computeStudentsStats)

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

7. 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.

8. Ojo con el trabajo colaborativo de github, solo veo 2 commits y son de @rxmstrd

Vamos con todooo, a salir ganadoras de estaaaa 💪