Skip to content

Add metro area codes and switch airport data to lxndrblz/Airports#6

Merged
rhinot merged 2 commits intomainfrom
pr-airport-metro-codes
Mar 22, 2026
Merged

Add metro area codes and switch airport data to lxndrblz/Airports#6
rhinot merged 2 commits intomainfrom
pr-airport-metro-codes

Conversation

@rhinot
Copy link
Copy Markdown
Collaborator

@rhinot rhinot commented Mar 22, 2026

Summary

  • Rewrites scripts/build-airport-map.mjs to fetch airports.csv + citycodes.csv from lxndrblz/Airports at build time — no local file or airport-timezone devDependency needed; OurAirports approach preserved in comments as backup
  • Regenerates netlify/airport-map-data.json: 9,803 airports + 27 metro codes + 5 derived = 9,835 total entries
  • Adds metro area code support (NYC, LON, TYO, PAR, CHI, etc.) — overrides in common.js trimmed to only the 4 not covered by the new data (TCI collision, QDF/QHO/QMI absent from dataset)
  • Updates homepage to document timezone abbreviation, airport code, and metro area code inputs
  • Fixes vertical centering on homepage (min-height: 100vh)
  • Cleans up .gitignore (adds .claude/, deno.lock, airports.csv)

Test plan

  • npm run build:airports fetches and regenerates airport-map-data.json successfully
  • Airport codes resolve correctly: SFO, JFK, LHR
  • Metro codes resolve correctly: NYC, LON, TYO, PAR, CHI
  • Overridden codes work: TCI (Tenerife), QDF (Dallas), QHO (Houston), QMI (Miami)
  • Homepage renders centered with updated copy

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 22, 2026

Deploy Preview for timezone-fyi ready!

Name Link
🔨 Latest commit 216e0de
🔍 Latest deploy log https://app.netlify.com/projects/timezone-fyi/deploys/69c0028b04b8900008a2d3aa
😎 Deploy Preview https://deploy-preview-6--timezone-fyi.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rhinot rhinot self-assigned this Mar 22, 2026
@rhinot rhinot requested review from alcor and liquidx March 22, 2026 14:38
@rhinot rhinot merged commit 478ec18 into main Mar 22, 2026
5 checks passed
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.

1 participant