Repository navigation
Deploy map through Cloudflare Pages Git integration instead of a workflow - #31
Conversation
…flow Remove the wrangler-based deploy workflow (it needed repository secrets that will not be added) and document the Pages project build settings. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PS44Q7WRmAzMJLhi44dzdo
|
No actionable comments were generated in the recent review. Walkthrough
Merge Risk:
|
coverage-api.airframes.io has no DNS record. The coverage service runs only on the staging cluster at staging-coverage.airframes.io, so use that host in the CI build args and the documented Pages settings. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PS44Q7WRmAzMJLhi44dzdo
Deploying airframes-coverage with
|
| Latest commit: |
20749b5
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f5061d6f.airframes-coverage.pages.dev |
| Branch Preview URL: | https://claude-project-thread-xlxhmp.airframes-coverage.pages.dev |


Requested by Kevin · project thread
Summary
Follow-up to #25. The map now deploys the same way the globe does: the Cloudflare Pages project
airframes-coveragein the Airframes account is connected to this repository and builds on every push and pull request. It is live at https://airframes-coverage.pages.dev/. That makes the wrangler-based workflow from #25 redundant, and it could never upload anyway because the repository has no Cloudflare secrets..github/workflows/deploy-map.yml.coverage-api.airframes.iowithstaging-coverage.airframes.ioin the README and in the Map CI build args.coverage-api.airframes.iohas no DNS record, so every build that used it showed an empty map. The coverage service currently runs only on the staging cluster, wherestaging-coverage.airframes.ioserves the API under/api/and the WebSocket at/ws.Pages project settings (as configured)
mapnpm run build:staticoutNODE_VERSION24NEXT_PUBLIC_COVERAGE_API_URLhttps://staging-coverage.airframes.ioNEXT_PUBLIC_WS_URLwss://staging-coverage.airframes.io/wsWithout the API variable the build points at
http://localhost:3002. Both are set for Production and Preview.Verification
mainsucceeded, and the page returns 200.staging-coverage.airframes.ioand nocoverage-api.airframes.ioorlocalhost:3002.https://staging-coverage.airframes.io/api/v1/statsand/api/v1/stationsreturn 200 JSON withaccess-control-allow-origin: *./api/v1/activity;NEXT_PUBLIC_WS_URLis not referenced by the current client code).Screenshots
None: this changes a workflow file, CI build args and the README only, with no UI change. The map's screenshots are in #25.
🤖 Generated with Claude Code
https://claude.ai/code/session_01PS44Q7WRmAzMJLhi44dzdo
Summary by CodeRabbit
mainand previews for pull requests, and documented the localhost fallback when API variables are unset.