search

transitmatters / t-performance-dash

TransitMatters performance visualizer for the MBTA
https://dashboard.transitmatters.org/
MIT License
51 stars 17 forks source link

add csv download link to speed, prediction accuracy, service, ridership #935

Closed cdwhitt closed 10 months ago

cdwhitt commented 10 months ago

Motivation

Commonly used widgets need a link to download a .csv file so people can use our data in research without needing to reach out to us #906

Changes

Screenshot 2024-01-11 at 12 32 58 PM

Screenshot 2024-01-13 at 5 22 18 PM Screenshot 2024-01-13 at 5 22 36 PM Screenshot 2024-01-13 at 5 22 57 PM Screenshot 2024-01-13 at 5 23 10 PM Screenshot 2024-01-13 at 5 23 24 PM Screenshot 2024-01-13 at 5 23 37 PM Screenshot 2024-01-13 at 5 23 47 PM Screenshot 2024-01-13 at 5 23 59 PM Screenshot 2024-01-13 at 5 24 16 PM Screenshot 2024-01-13 at 5 24 26 PM

Testing Instructions

  1. Load the app locally
  2. Widgets on the Line Overview page should have download links (Speed, Service, Ridership). Arrival Predictions should have a download button as well.
  3. Test by adjusting time frame on the top right, click download button for widget, ensure data is within time range
cdwhitt commented 10 months ago

I responded to probably half of the comments here, will finish up tomorrow (1/14). @skaplan-dev @idreyn

cdwhitt commented 10 months ago

Could you add some images/gifs and more information to the PR description?

It also looks like there were some moving around of ChartBorder. Were we using it inconsistently in places?

Also wanted to mention, this needed to be moved up a level as it caused some inconsistency when I added the DownloadButton to one of the widgets. I checked, and moving it didn't affect anything else.