midudev / covid-vacuna

App para ver el estado de la vacunación COVID-19 en España
https://covid-vacuna.app
Apache License 2.0
365 stars 126 forks source link

feat: add filters for charts #61

Closed Miangame closed 3 years ago

Miangame commented 3 years ago

Hola Midu! Dejo por aquí mi pequeña aportación.

He añadido que el filtro funcione también en las gráficas, para ello he tenido que crear otro script de lectura de archivos, pero he querido dejar el original.

Un saludo y muchas gracias por el gran curro! 😄

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/midudev/covid-vacuna/mq5j9vix7
✅ Preview: https://covid-vacuna-git-fork-miangame-feature-added-filter-for-charts.midudev.vercel.app

midudev commented 3 years ago

Gracias por el aporte Miguel Ángel. Veo que cuando cambio el filtro, las fechas de las gráficas no dejan de repetirse.

¿Le puedes echar un vistazo?

image

vercel[bot] commented 3 years ago

@Miangame is attempting to deploy a commit to a Personal Account owned by @midudev on Vercel.

@midudev first needs to authorize it.

Miangame commented 3 years ago

Perdona la demora @midudev, no he podido arreglarlo antes por temas de trabajo. Ya he arreglado el problema de las fechas que me comentaste. Además he quitado la escala fija que había en las gráficas para que los datos escalen mejor, ya que si hay una ccaa con un número muy bajo, se vería de esta manera:

Screenshot 2021-02-11 at 20 16 27

Saludos!

othermore commented 3 years ago

+1 a este cambio (no lo he probado, pero por lo que hace según la descripción). Estaría super bien ver el gráfico de vacunas entregadas y administradas filtrado por comunidad, lo mismo que el resto de información.