Skip to content

feat: license tag filtering#83

Merged
Alessandro100 merged 2 commits intomainfrom
feat/1567-license-tag-filtering
Mar 23, 2026
Merged

feat: license tag filtering#83
Alessandro100 merged 2 commits intomainfrom
feat/1567-license-tag-filtering

Conversation

@Alessandro100
Copy link
Contributor

Summary:

closes MobilityData/mobility-feed-api#1567

Ability to filter licenses by tag

Expected behavior:

  1. When you hover over the license, it will display all the license tags
  2. You can select a license tag which will change the URL and display the feeds with the associated license tag
  3. When the license tag is selected, it will show on the card to give user assurance that the filter is working

Testing tips:

Go on the feeds page and play around with the filter

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

  • Run the unit tests with yarn test 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)
Screenshot 2026-03-23 at 13 17 05 Screenshot 2026-03-23 at 13 15 38 Screenshot 2026-03-23 at 13 15 54 Screenshot 2026-03-23 at 13 16 00

@Alessandro100 Alessandro100 requested a review from Copilot March 23, 2026 17:21
@Alessandro100 Alessandro100 self-assigned this Mar 23, 2026
@vercel
Copy link

vercel bot commented Mar 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mobilitydatabase-web Ready Ready Preview, Comment Mar 23, 2026 5:33pm

Request Review

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

@github-actions
Copy link

*Lighthouse ran on https://mobilitydatabase-byxly7a5f-mobility-data.vercel.app/ * (Desktop)
⚡️ HTML Report Lighthouse report for the changes in this PR:

Performance Accessibility Best Practices SEO
🟢 94 🟢 96 🟢 100 🟢 100

*Lighthouse ran on https://mobilitydatabase-byxly7a5f-mobility-data.vercel.app/feeds * (Desktop)
⚡️ HTML Report Lighthouse report for the changes in this PR:

Performance Accessibility Best Practices SEO
🟠 87 🟠 87 🟢 96 🟢 100

*Lighthouse ran on https://mobilitydatabase-byxly7a5f-mobility-data.vercel.app/feeds/gtfs/mdb-2126 * (Desktop)
⚡️ HTML Report Lighthouse report for the changes in this PR:

Performance Accessibility Best Practices SEO
🟠 74 🟢 94 🟢 96 🟢 100

*Lighthouse ran on https://mobilitydatabase-byxly7a5f-mobility-data.vercel.app/feeds/gtfs_rt/mdb-2585 * (Desktop)
⚡️ HTML Report Lighthouse report for the changes in this PR:

Performance Accessibility Best Practices SEO
🟠 82 🟠 83 🟢 100 🟢 100

*Lighthouse ran on https://mobilitydatabase-byxly7a5f-mobility-data.vercel.app/feeds/gbfs/gbfs-flamingo_porirua * (Desktop)
⚡️ HTML Report Lighthouse report for the changes in this PR:

Performance Accessibility Best Practices SEO
🟢 90 🟢 94 🟢 96 🟢 100

@Alessandro100 Alessandro100 merged commit 60b8de5 into main Mar 23, 2026
4 checks passed
@Alessandro100 Alessandro100 review requested due to automatic review settings March 23, 2026 22:03
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.

UI: Filter by license family

2 participants