Skip to content

fix(landing): disable analytics outside production - #642

Merged
owjs3901 merged 1 commit into
mainfrom
owjs3901/fix-analitics
Aug 27, 2026
Merged

fix(landing): disable analytics outside production#642
owjs3901 merged 1 commit into
mainfrom
owjs3901/fix-analitics

Conversation

@owjs3901

Copy link
Copy Markdown
Contributor

Summary

  • Initialize Google Tag Manager only on devup-ui.com.
  • Remove the no-JavaScript GTM fallback iframe.
  • Add localhost regression coverage for blocked analytics requests.

Validation

  • bun tsc --noEmit -p apps/landing/tsconfig.json
  • bun eslint apps/landing/src/app/layout.tsx e2e/helpers.ts e2e/landing-build-integrity.spec.ts e2e/landing-zero-runtime.spec.ts
  • bunx playwright test e2e/landing-build-integrity.spec.ts --grep "Google Tag Manager is disabled"
  • bunx playwright test e2e/docs-pages.spec.ts --grep "header has Docs link"

Note: the repository pre-commit coverage stage could not complete in this Windows environment because Cargo reported a closed-pipe/cache dependency error for zerocopy_derive.

@github-actions

Copy link
Copy Markdown
Contributor

Changepacks

@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@owjs3901
owjs3901 merged commit b157260 into main Aug 27, 2026
4 checks passed
@owjs3901
owjs3901 deleted the owjs3901/fix-analitics branch August 27, 2026 09:10
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