Skip to content

revert: remove the Cloudflare Web Analytics beacon - #798

Merged
thomaspinder merged 1 commit into
mainfrom
revert/cloudflare-beacon
Sep 30, 2026
Merged

thomaspinder merged 1 commit into
mainfrom
revert/cloudflare-beacon

Conversation

@thomaspinder

Copy link
Copy Markdown
Collaborator

Motivation

The Cloudflare Web Analytics beacon added in #796 does not work. The docs are served by GitHub Pages, outside the Cloudflare zone, and the quantclimate.com Web Analytics site does not accept beacons from this hostname. In the browser, each page view logs this error and records nothing:

Access to XMLHttpRequest at 'https://cloudflareinsights.com/cdn-cgi/rum' from origin 'https://gpjax.quantclimate.com' has been blocked by CORS policy

Solution

Revert the change to docs/conf.py. Analytics will come from serving the docs through Cloudflare (a follow-up PR), where Cloudflare adds the beacon itself and no code in the repo is necessary.

Verification

docs/conf.py is byte-identical to its state before #796.

🤖 Generated with Claude Code

https://claude.ai/code/session_015fagcMeo1LmLG2Dg3NybmQ

This reverts commit ccc299b.

Cloudflare rejects the beacon on this hostname: the docs are served
by GitHub Pages, outside the Cloudflare zone, and the quantclimate.com
Web Analytics site does not accept it. Each page view logged a CORS
error and recorded nothing. Analytics will come from serving the docs
through Cloudflare instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015fagcMeo1LmLG2Dg3NybmQ
@thomaspinder
thomaspinder enabled auto-merge (squash) September 30, 2026 20:12
@github-actions github-actions Bot added documentation Improvements or additions to documentation size/s labels Sep 30, 2026
@github-actions

Copy link
Copy Markdown

📖 Docs preview: https://pr-798--endearing-crepe-c2d5fe.netlify.app

Smoke render — the expensive notebooks run with reduced budgets, so
figures are not publication fidelity. /render-mode.txt says smoke.

@thomaspinder
thomaspinder merged commit 286110e into main Sep 30, 2026
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant