Skip to content

Feed Submission November 27 #1168

Merged
emmambd merged 8 commits intomainfrom
import/november-27-eu-feeds
Nov 28, 2025
Merged

Feed Submission November 27 #1168
emmambd merged 8 commits intomainfrom
import/november-27-eu-feeds

Conversation

@ianktc
Copy link
Copy Markdown
Contributor

@ianktc ianktc commented Nov 28, 2025

},
"urls": {
"direct_download": "https://www.gradski-parking.hr/upload/stranice/2022/08/2022-08-30/89/gtfs.zip",
"latest": "https://storage.googleapis.com/storage/v1/b/mdb-latest/o/hr-gradski-parking-gtfs-2915.zip?alt=media",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verified as official?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah so I went to their site here https://www.gradski-parking.hr/stranice/javni-gradski-prijevoz/86.html and it does seem official (as far as I can tell). A bit odd maybe because "parking" is in the name..

@emmambd
Copy link
Copy Markdown
Contributor

emmambd commented Nov 28, 2025

@ianktc Looks great! Only one comment from me. I think we should reach out to Passenger after this PR is merged, email them to ask if these feeds are in fact official from the agencies.

@emmambd
Copy link
Copy Markdown
Contributor

emmambd commented Nov 28, 2025

@ianktc Looks like it failed the integration tests - maybe a duplicate URL somewhere?

}
},
"urls": {
"direct_download": "https://www.blackpooltransport.com/open-data/network/current?format=gtfs",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks like the official URL, not the passenger one. Thoughts?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah so basically I had the wrong URLs for two of the feeds, but should be corrected now

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah failed again.. it was an uncaught duplicate (mdb-1938). Fixed and letting checks run again

@emmambd emmambd merged commit 4f4f110 into main Nov 28, 2025
10 checks passed
@emmambd emmambd deleted the import/november-27-eu-feeds branch November 28, 2025 17:34
@ianktc ianktc self-assigned this Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants