Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

project(data-lovers): Actualizar set de datos y agregar researchs de usuarios #943

Merged
merged 16 commits into from
Mar 11, 2020

Conversation

betsyvies
Copy link
Contributor

@betsyvies betsyvies commented Feb 21, 2020

Primero comentarles el motivo del pull request, como equipo de CC estuvimos trabajando para repensar el proyecto data-lovers y su ejecución, ya que, vimos que este proyecto es el que más frustra a las estudiantes.

Cuando empezamos a investigar porque pasa esto nos dimos cuenta de que varios de los set de datos, específicamente steam, world bank, injuries y patient son innecesariamente complejos, la mayoría de los datos son ceros o vacíos, lo cual dificulta a las estudiantes acceder a ellos y mostrarlos en su página. Al quitarlos y probar data-lover 2.0 con la generación LIM011 comprobamos que, las estudiantes se sentían motivadas al final de proyecto y la gran mayoría logro cumplir con los objetivos de aprendizaje del proyecto.

Agregando a lo anterior, también en nuestra investigación nos dimos cuenta que las estudiantes invierten mucho tiempo en la investigación con los usuarios, que en crear sus HU y trabajar con estas. Por eso al crear data-lover 2.0 agregamos por cada set de datos un research que el equipo hizo previamente. Esto les permitió tener una visión más clara de lo que tenían que hacer, mayor organización y por ende mejor trabajo en equipo.

En conclusión, nuestra experiencia sugiere que quitemos los set de datos mencionados anteriormente, que nos quedemos con lol y pókemon, también que agreguemos unos nuevos que aporten al aprendizaje de las estudiantes, pues con dos no hay variedad. El data-set rick and morty no lo he sacado porque va en linea con lo que estamos proponiendo, en cuanto al research aún no esta creado. Sería genial si algún coach que haya trabajado de cerca con este set nos apoyará con el reaserch, toda ayuda esta bienvenida.

Equipo CC: @CaroLaboratoria @diegovelezg @lalogf @developerVilchez @RuthSalvador @MarciaMillanao 🌟

@Laboratoria/bootcamp-coaches @juanjordan @lupomontero

@betsyvies betsyvies changed the title project(data-lovers): Actualizar set de datos project(data-lovers): Actualizar set de datos y agregar researchs de usuarios Feb 21, 2020
Copy link
Contributor

@juanjordan juanjordan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hola @betsyvies, gracias por enviar este PR, he leído detenidamente las razones de los cambios propuestos y creo que son sensatas las conclusiones a las que han llegado. No obstante, estoy notando que a parte de cambios relacionados a lo propuesto existen otros relacionados a la funcionalidad de la aplicación web esperada, como la capacidad de filtrar, ordenar y/o hacer estadísticas sobre los set de datos presentados. ¿hay alguna conclusión a la que se llego que hablaba de este tema?

También veo que el research de Rick & Morty está vacío y el CI está indicando algunos detalles, te recomiendo correr npm test de manera local.

Dejo comentarios al respecto y a un cambio de la identidad gráfica. ¡Sigamos conversando, gracias!

projects/02-data-lovers/README.md Outdated Show resolved Hide resolved
como la diseñaste. Tendrás un tiempo limitado para hackear, así es que deberás
priorizar.
**No** es necesario que construyas la interfaz exactamente como la diseñaste.
No tienes tiempo ilimitado para trabajar, así es que deberás priorizar.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No sé si necesitamos una frase así que parece una doble negación "No tienes tiempo ilimitado", propondría agregar algo como: "Tu tiempo de hacking es escaso, así que deberás priorizar". ¿Qué les parece?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Estoy de acuerdo.

4. Ser _responsive_, es decir, debe visualizarse sin problemas desde distintos
1. Mostrar la data en una interfaz: puede ser un card, una tabla, una lista,
etc.
2. Permitir al usuario interactuar para obtener la infomación que necesita.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

¿Por qué estaríamos dejando de indicar que el usuario puede filtrar y ordenar la data? y ¿por qué está comentado líneas abajo? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Creímos que era redundante, pero revisándolo ahora no está demás recordar los métodos que deben utilizar.

projects/02-data-lovers/README.md Show resolved Hide resolved
projects/02-data-lovers/README.md Outdated Show resolved Hide resolved
projects/02-data-lovers/src/data/lol/README.md Outdated Show resolved Hide resolved
@juanjordan juanjordan added the on hold No es prioridad ni urgente label Mar 9, 2020
@juanjordan
Copy link
Contributor

También veo que el research de Rick & Morty está vacío

Este PR está on-hold hasta que tengamos contenido para el research de Rick and Morty, @betsyvies estará preguntando al equipo de UX al respecto.

@juanjordan juanjordan removed the on hold No es prioridad ni urgente label Mar 11, 2020
Copy link
Contributor

@juanjordan juanjordan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@juanjordan juanjordan merged commit 52acf7a into Laboratoria:master Mar 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants