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

Adding GTFS Digest to Index.md #256

Merged
merged 3 commits into from
Dec 20, 2023
Merged

Adding GTFS Digest to Index.md #256

merged 3 commits into from
Dec 20, 2023

Conversation

eliasmbd
Copy link
Contributor

Adding the link to GTFS Digest on GTFS.org main page.

Adding the link to GTFS Digest on GTFS.org main page.
@eliasmbd eliasmbd requested a review from isabelle-dr December 19, 2023 16:57
Copy link
Contributor

@isabelle-dr isabelle-dr left a comment

Choose a reason for hiding this comment

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

I like the idea of adding some community resources directly to the landing page!

The "community" resources currently appear in two locations

  1. https://gtfs.org/schedule/#getting-help-community
  2. https://gtfs.org/resources/community/

In the intention of making information accessible, I'd like to propose that we don't add a third and instead, we move the whole Getting Help and Community section to the landing page and add the GTFS Digest as the first bullet point.

We can certainly use communication efforts to ensure the community is aware that this now exists! As a GTFS user visiting the site to look for community resources, I'd want to see all of them together and pick which the one(s) works best for me.

Additionally, we can add it to the awesome transit list, which is the source for the Resources section, so that it appears in both locations 🙂.

@eliasmbd
Copy link
Contributor Author

@isabelle-dr I understand your concern and I think we should definetly bring this up during our 3 day hackathon in january. This push is a temporary fix until then.

Copy link
Contributor

@isabelle-dr isabelle-dr left a comment

Choose a reason for hiding this comment

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

Approving this PR before I go on holidays so you can merge it this week.
Leaving it up to you to make modifications or not based on our discussion :)

@isabelle-dr
Copy link
Contributor

@eliasmbd I believe this is your first PR on gtfs.org 🥳, so heads-up that we maintain a translation backlog that we keep up to date at all times.

People that merge a PR are responsible for updating the translation backlog (example), so don't forget to add this PR when you merge!

@eliasmbd eliasmbd merged commit 864d704 into main Dec 20, 2023
@eliasmbd eliasmbd deleted the eliasmbd-patch-1 branch December 20, 2023 19:43
@eliasmbd eliasmbd mentioned this pull request Dec 20, 2023
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.

2 participants