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 contributing web guide after Navitia removal #976

Open
Lastaapps opened this issue Dec 15, 2024 · 5 comments
Open

Update contributing web guide after Navitia removal #976

Lastaapps opened this issue Dec 15, 2024 · 5 comments
Labels
bug 🐞 A functional defect or unexpected behavior. priority 🚨️ This issue will be most probably addressed before the next release.

Comments

@Lastaapps
Copy link

Describe the bug
The main project webpage has a contributing guide describing how a new provider can be added. What I learned from release notes is that Navitia API got shut down, therefore if I understand it correctly, the guide has to be updated significantly as it's mainly based on contacting the official Navitia host in France. What is the preferred way now? Self-hosting? public-transport-enables is also not very descriptive either. I think the guide should contain the new preferred way to get GTFS into the app.

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://transportr.app/contribute/#add_region
  2. Read

Expected behavior
Clear, up-to-date guidance on how to add a new provider using GTFS.

Versions:

  • webpage as of 15. December 2024
@Lastaapps Lastaapps added the bug 🐞 A functional defect or unexpected behavior. label Dec 15, 2024
@Altonss
Copy link
Collaborator

Altonss commented Dec 15, 2024

The main project webpage has a contributing guide describing how a new provider can be added. What I learned from release notes is that Navitia API got shut down, therefore if I understand it correctly, the guide has to be updated significantly as it's mainly based on contacting the official Navitia host in France.

The webpage code has been updated here: grote/Transportr-website#15, but just needs to be pushed/published by @grote :)

For GTFS integration, you can read about it here #907 (comment) and also here #954 ; but for now the TL;DR is : there is currently no way to integrate GTFS into the app.

@Altonss Altonss closed this as completed Dec 15, 2024
@Altonss Altonss added duplicate ➿ This ticket already exists. and removed duplicate ➿ This ticket already exists. labels Dec 15, 2024
@Altonss Altonss reopened this Dec 15, 2024
@Altonss
Copy link
Collaborator

Altonss commented Dec 15, 2024

I'll keep this issue open until the new website content has been published :)

@Altonss Altonss added the priority 🚨️ This issue will be most probably addressed before the next release. label Dec 15, 2024
@Lastaapps
Copy link
Author

Nice, thank you for the summary and clarification.

@grote
Copy link
Owner

grote commented Dec 16, 2024

check now please

@Lastaapps
Copy link
Author

A quick sentence about that and why GTFS is not currently supported would be nice. Also, the examples in the next paragraph Adding it to Transportr still point to pull requests that add a Navitia source to the enabler (AbstractNavitiaProvider is used). Not sure how much does the process differ for the other data formats.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 A functional defect or unexpected behavior. priority 🚨️ This issue will be most probably addressed before the next release.
Projects
None yet
Development

No branches or pull requests

3 participants