Skip to content

fix(landing): restore homepage view tracking - #6260

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix/landing-page-metrics
Aug 4, 2026
Merged

fix(landing): restore homepage view tracking#6260
waleedlatif1 merged 1 commit into
stagingfrom
fix/landing-page-metrics

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Summary

  • remount the landing analytics tracker on the homepage
  • add regression coverage for the tracker composition

Type of Change

  • Bug fix

Testing

  • bun run test 'app/(landing)/landing.test.tsx'
  • bun run lint
  • full ship audit suite

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Aug 4, 2026 7:39pm

Request Review

@cursor

cursor Bot commented Aug 4, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Analytics-only change on the marketing landing page with a focused test; no auth, data, or core workflow impact.

Overview
Restores homepage analytics by rendering LandingAnalytics at the top of the landing main tree so the client tracker mounts again and fires landing_page_viewed on visit.

Adds a node SSR regression test that stubs landing sections and asserts the analytics marker appears exactly once in static markup, guarding against duplicate or missing composition.

Reviewed by Cursor Bugbot for commit e1b35c3. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR restores homepage analytics by mounting the existing LandingAnalytics component in the landing-page composition.

  • Adds the analytics tracker once within the homepage root.
  • Adds a server-rendered regression test that verifies the tracker remains in the composition exactly once.

Confidence Score: 5/5

The PR appears safe to merge with no actionable correctness or security issues identified.

The homepage now mounts the existing tracker exactly once, and the added regression test directly covers that composition without altering other landing-page behavior.

Important Files Changed

Filename Overview
apps/sim/app/(landing)/landing.tsx Mounts the existing client-side landing analytics tracker once alongside the homepage content.
apps/sim/app/(landing)/landing.test.tsx Adds focused composition coverage that detects removal or duplicate rendering of the tracker.

Reviews (1): Last reviewed commit: "fix(landing): restore homepage view trac..." | Re-trigger Greptile

@waleedlatif1
waleedlatif1 merged commit a19ae16 into staging Aug 4, 2026
24 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/landing-page-metrics branch August 4, 2026 19:45
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.

2 participants