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

Feat: update features for validator 6.0 #771

Merged
merged 3 commits into from
Oct 22, 2024
Merged

Conversation

Alessandro100
Copy link
Contributor

closes #733
Summary:

Update the Feature keys and links in the frontend to reflect the upcoming validator 6.0 release while also supporting previous versions

Expected behavior:

When going to the feed detail page of any feed, under the "Feature" section, you should be able to click on any of them and it will direct you to the correct gtfs.org page

Testing tips:

Go to any feed detail page, when you click on a feature make sure it maps correctly based on https://github.com/MobilityData/gtfs-validator/blob/master/docs/FEATURES.md

For the deprecated features (as of 6.0) they should navigate to the corresponding updated feature page

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with ./scripts/api-tests.sh to make sure you didn't break anything
  • Add or update any needed documentation to the repo
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

@Alessandro100 Alessandro100 self-assigned this Oct 22, 2024
@@ -104,31 +118,32 @@ export const DATASET_FEATURES: DatasetFeatures = {
fileName: 'levels.txt',
linkToInfo: 'https://gtfs.org/getting_started/features/pathways/#levels',
},
'Traversal Time': {
'In-station traversal time': {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: revisit the upper / lower case of the spelling of this key

@Alessandro100
Copy link
Contributor Author

Decided to go with manually adding the linkToInfo for each feature over generating it with feature + feature group because

  • Gives us more flexibility in the future if the url pattern changes per feature
  • Less complexity
  • Not many features so automating this process doesn't yield massive gains

Copy link

Preview Firebase Hosting URL: https://mobility-feeds-dev--pr-771-xr0pe191.web.app

Copy link
Member

@davidgamez davidgamez left a comment

Choose a reason for hiding this comment

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

LGTM

fileName: 'frequencies.txt',
linkToInfo:
'https://gtfs.org/getting_started/features/base_add-ons/#frequency-based-service ',
},
};

// DEPRECATED FEATURES
Copy link
Member

Choose a reason for hiding this comment

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

👍

@Alessandro100 Alessandro100 merged commit 8ffdbea into main Oct 22, 2024
4 checks passed
@Alessandro100 Alessandro100 deleted the feat/733-update-features branch October 22, 2024 15:21
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.

Add Ids to gtfs Features
2 participants