Skip to content

Add Talvess analytics - #60

Merged
radiumcoders merged 1 commit into
mainfrom
talvess-analytics
Nov 1, 2026
Merged

radiumcoders merged 1 commit into
mainfrom
talvess-analytics

Conversation

@radiumcoders

@radiumcoders radiumcoders commented Nov 1, 2026 •

Copy link
Copy Markdown
Owner

Loads the Talvess tracker site-wide from app/layout.tsx via next/script (afterInteractive), alongside the existing Tracwell analytics. No CSP exists, so no header changes.

Checks: pnpm typecheck, eslint, and next build pass; the built home page includes the tracker once.

Summary by CodeRabbit

  • Other
    • The site now loads an additional external resource after the page becomes interactive. This update does not change the visible interface or add new controls. The resource loads as part of the page experience, while existing navigation and content remain unchanged. No other user-facing changes are included.

@vercel

vercel Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
23rd-dev Ready Ready Preview Nov 1, 2026 12:34pm UTC

@coderabbitai

coderabbitai Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Walkthrough

The app layout now loads an external analytics script with a specified website ID after the page becomes interactive.

Changes: Analytics Script Integration

Layer / File(s) Summary
Load analytics script
app/layout.tsx
The layout imports Next.js Script and loads the external analytics script with a website ID using the afterInteractive strategy.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Feature

Suggested reviewers: cursoragent

Merge Risk: Moderate · up to 294c5

Visitors are subject to undisclosed analytics collection, and development visits can affect production metrics. Resolve the privacy disclosure and applicable consent requirements before merging.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@radiumcoders
radiumcoders merged commit e59481f into main Nov 1, 2026
3 of 4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 294c50e0 Deployed Nov 1, 2026 by vercel[bot]
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