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

Update Metro Transit (Wisconsin) feeds to use new URLs [SOURCES] #494

Merged
merged 16 commits into from
Aug 29, 2024

Conversation

djmeier-wisc
Copy link
Contributor

Hi there,

I am looking to update the realtime information for Madison Wisconsin's Metro Transit. They recently upgraded their AVL system, and the URLs changed, unfortunately without redirects in place. Their static data URL has not changed.

Please let me know if I missed any details.

@CLAassistant
Copy link

CLAassistant commented Aug 26, 2024

CLA assistant check
All committers have signed the CLA.

@emmambd
Copy link
Contributor

emmambd commented Aug 27, 2024

Thank you so much @djmeier-wisc for this contribution!

We don't change the URL for feeds, so that there's a consistent definition of what a feed is over time. Could you please create new feeds for these new URLs, and set the status to the old feeds as deprecated? You can add a redirect from the old feed to the new feed IDs. More details on the status and redirect fields in the schema here.

Let me know if you have any questions about this process.

@djmeier-wisc
Copy link
Contributor Author

Thanks @emmambd! My bad. Let me know if you see any issues with these changes, or if the ID is already in use

Copy link
Contributor

@emmambd emmambd left a comment

Choose a reason for hiding this comment

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

Thanks for this great contribution @djmeier-wisc! I just updated the IDs to make sure the integration tests passed.

@emmambd emmambd merged commit e826b33 into MobilityData:main Aug 29, 2024
5 of 6 checks passed
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.

3 participants