EXP-1609: Migrate deployment to GitHub Pages - #1862
Merged
Merged
Conversation
- deploy.yml: build on main + publish to GitHub Pages (Node 24, npm ci with NPM_TOKEN against the ADO feed, build env vars via repo secrets/variables) - pr.yml: build-only PR check (replaces Vercel preview deployments) - static/CNAME for docs.codat.io - Remove @docusaurus/plugin-vercel-analytics plugin + dependency (lockfile edited by hand; npm ci in CI validates it) - vercel.json intentionally untouched: Vercel still serves prod from main until decommission, and it carries the live CORS headers Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
pmckinney-codat
marked this pull request as ready for review
July 27, 2026 12:11
…ets, not variables) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The Sign-in click is already tracked via Amplitude; the Vercel track() call was the only remaining usage and broke the build (and the Vercel preview) once the dependency was removed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
GTM_ID and the feature flags are public values; vars keep them visible in the UI and unmasked in logs. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Link check results for preview deployment (https://codat-docs-git-exp-1609-github-pages-migration-codat.vercel.app): |
pmckinney-codat
enabled auto-merge
July 27, 2026 12:25
tom-binnington-codat
approved these changes
Jul 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Migrates docs.codat.io deployment from Vercel to GitHub Pages (EXP-1609, part of the Vercel migration workstream). Follows the pattern proven on legal.codat.io (codatio/legal#74) and the engineering blog.
.github/workflows/deploy.yml— on push tomain: Node 24,npm ci(ADO feed viaNPM_TOKEN),docusaurus build, publish to GitHub Pages. Nofetch-depth: 0needed — this site doesn't useshowLastUpdateAuthor/Time..github/workflows/pr.yml— build-only PR check. This replaces Vercel preview deployments (tradeoff acked by the docs team).static/CNAME—docs.codat.iofor the custom domain.@docusaurus/plugin-vercel-analytics(plugin + dependency). Lockfile was edited by hand (leaf dependency;@vercel/oidcstays — it belongs to@ai-sdk/gateway); thenpm ciin this PR's own check validates it.Deliberately NOT in this PR
vercel.jsonstays — Vercel keeps auto-deploying prod frommainuntil decommission, and it carries the live CORS headers on/updates/rss.xml. It gets deleted in a cleanup PR after cutover.vercel.jsonredirect (/docs/angular/your-first-app/2-taking-photos) was not ported toplugin-client-redirects: neither source nor target exists anywhere in the repo — it's an Ionic-template leftover.Before merging
NPM_TOKEN,ZENDESK_KEY,AMPLITUDE_API_KEY,GTM_ID,FEATURE_DEV_FLAG,FEATURE_NEW_PRODUCTS_FLAG(all captured from Vercel;GA_IDalso copied over but unused by the code)members_can_create_public_pagesenabled (needed at site-creation time)/updates/rss.xmlCORS consumer: resolved — the only consumer is portal-ui's "What's New" fetch (app.codat.io), a plainaxios.getwith no custom headers since Datadog RUM was removed from portal-ui in Sep 2025 (PEP-141). No preflight is sent; GitHub Pages' defaultaccess-control-allow-origin: *suffices. The x-datadog allow-headers in vercel.json are dead config.After merge, the site at
codatio.github.io/codat-docswill look broken with a "wrong baseUrl" banner — expected, it's built for docs.codat.io. Then set the custom domain in Pages settings.🤖 Generated with Claude Code