Skip to content

Release: codebar analytics fixes - #132

Merged
StanBarrows merged 1 commit into
productionfrom
main
Oct 25, 2026
Merged

StanBarrows merged 1 commit into
productionfrom
main

Conversation

@StanBarrows

Copy link
Copy Markdown
Contributor

Summary

  • Load the codebar analytics tracker with crossorigin="anonymous", so the browser ignores Cloudflare's __cf_bm cookie on analytics.codebar.ch (no third-party cookie, Lighthouse best practices stays at 100).
  • Close the remaining tracking gap of this project (see commit message: layout without Fathom, or APP_ENV=staging on the serving environment).

Test plan

  • Analytics partial/component tests, new case for the fixed layout or environment
  • CI

🤖 Generated with Claude Code

… credentials

zunscan.codebar.ch has its own layout without Fathom, so the tracker was
missing there; it now includes the codebar analytics partial too.
crossorigin="anonymous" fetches script.js without credentials, so the
browser ignores Cloudflare's __cf_bm cookie on analytics.codebar.ch
instead of storing it as a third-party cookie.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@StanBarrows
StanBarrows merged commit 7f2f8ec into production Oct 25, 2026
6 of 7 checks passed
StanBarrows pushed a commit that referenced this pull request Oct 25, 2026
Larastan analyses the tests and does not know $this inside the Pest
closure (4 errors in ci larastan on #132). Use Pest\Laravel\get() and
withoutVite() instead, as the rest of the suite does.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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