Skip to content

feat: revalidation cron changed to gbfs#95

Draft
Alessandro100 wants to merge 1 commit intomainfrom
feat/67-revalidate-cron-updated
Draft

feat: revalidation cron changed to gbfs#95
Alessandro100 wants to merge 1 commit intomainfrom
feat/67-revalidate-cron-updated

Conversation

@Alessandro100
Copy link
Copy Markdown
Contributor

Summary:

part 1/3 of #67

Changes the revalidation cron from revalidating everything to just gbfs feeds. gtfs and gtfs_rt are going to be revalidated outside of this system

Expected behavior:

From monday to saturday at 4am utc and on sunday 7am utc we will revalidate all the gbfs feed cache. From mon-sat is the estimated time of validation finishing in GCP, and on sunday is the estimated time of geolocation finishing

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)

@Alessandro100 Alessandro100 requested a review from cka-y April 1, 2026 17:28
@Alessandro100 Alessandro100 self-assigned this Apr 1, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

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

Project Deployment Actions Updated (UTC)
mobilitydatabase-web Ready Ready Preview, Comment Apr 1, 2026 5:28pm

Request Review

@cka-y
Copy link
Copy Markdown
Contributor

cka-y commented Apr 1, 2026

@Alessandro100 this shouldn't be merged before #67 is completed right?

@Alessandro100
Copy link
Copy Markdown
Contributor Author

yeah this shouldn't be merged before the rest of #67 is complete

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

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

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

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

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

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

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

*Lighthouse ran on https://mobilitydatabase-hpzqfzklf-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
🟠 88 🟠 83 🟢 96 🟢 100

*Lighthouse ran on https://mobilitydatabase-hpzqfzklf-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
🟠 83 🟢 94 🟢 96 🟢 100

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