[Marc Folch] project-IV-sql-tableau

[marcfolchp]

https://github.com/marcfolchp/project4

[ta-data-bcn]

Holaa Marc! 🚀

Muchas felicidades por tu proyecto!

Un poco de feedback

Respecto a la organización del repositorio 📚

• El repo está bien estructurado, pero no veo gitignore?
• Por lo demás en este caso no son muchos files, pero para el proyecto final es súper recomendable tener carpetas para todo (notebooks, funciones, otros archivos tipo sql, etc.)
• Por lo demás benne!

El README.md 📖

• El Readme está muy bien, explicas los procesos y los archivos que hay en tu repo, el funcionamiento de éstos y muy guay también el enfoque de dear reader!
• Te recomiendo explicar un poco más dentro del readme, por ejemplo las queries de SQL las podrías haber explicado un poco mejor, la parte de tableau también.
• Explicación de las librerías y lenguajes de programación utilizados.
• El objetivo del análisis es sacar insights y findings después de procesar la data, y esto falta un poco en el readme. Falta un poco de conclusión de explicar las cosas que se han descubierto después del análisis

Code,SQL, Tableau 👩‍💻

• Muy tops tus funciones de Selenium!!! Felicidades!!!!
• Única cosa que veo que me da un poco de TOC son los imports, se ponen siempre fuera de las funciones!!
• También es muy top y lo hace más fácil de entender que pongas minicomentarios explicando qué hace cada cosa dentro de las funciones!
• Tus queries de SQL están bien también, creo que has pasado una parte bastante grande del tiempo en Selenium así que igual esta parte se te ha quedado un poco cortita.
• El Tableau también está muy bien, los dos primeros Dashboards son super explicativos y están super bien las pages para poder centrarte en los distintos establecimientos.
• El último Dashboard no me gusta ya que parece que sea un HATER del Mcdonalds, y NO ES VERDAD.