-
Notifications
You must be signed in to change notification settings - Fork 18
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
Adding the Mattapan trolley #956
Conversation
all of the frontend stuff lgtm in the beta, I'll wait to approve until we have the data sorted though |
Data is missing for May need to contact the MBTA to figure out why, or dig into LAMP data to figure out if there's a stop id that doesn't exist in the v3 api |
Missing data is a known issue. Doesn't block releasing, but does require a notice and maybe to disable selecting it in the UI |
Requires transitmatters/data-ingestion#117 now to merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did a quick glance at the gtfs for the stop ids but it looks good 👍
Motivation
Adds the Mattapan Trolley to the dashboard
We already have the data for it and the performance API supports it
Need to make sure some of the stops are correct, whoever reviews please sanity check
stations.json
stop mapsChanges
Adds the Mattapan Trolley to the dashboard using Subway presets and a few disabled pages
Testing Instructions
Test on beta, paying attention to weirdness