Skip to content

[codex] Remove product-app marketing pixels#2716

Merged
claudfuen merged 1 commit into
mainfrom
codex/app-marketing-pixels-gate
Apr 30, 2026
Merged

[codex] Remove product-app marketing pixels#2716
claudfuen merged 1 commit into
mainfrom
codex/app-marketing-pixels-gate

Conversation

@claudfuen
Copy link
Copy Markdown
Contributor

@claudfuen claudfuen commented Apr 30, 2026

Summary

Removes product-app marketing tracking from app.trycomp.ai entirely. The app keeps PostHog product analytics, while GTM, Google Ads/GA4 conversion tracking, LinkedIn Insight, and LinkedIn conversion APIs stay out of the product app.

What changed

  • Removed the product-app GTM render path and @next/third-parties.
  • Removed the product-app LinkedIn Insight component/render path.
  • Removed Google Ads, GA4 Measurement Protocol, LinkedIn Insight, and LinkedIn CAPI env vars from the app env schema, Docker build args, and self-hosting docs.
  • Simplified client/server tracking helpers to send only to PostHog.
  • Removed the app-side generic onboarding conversion event and ad conversion sends.
  • Updated tracking docs to be PostHog-only.
  • Clarified that app-side purchase_completed events are product analytics only; paid-media revenue, customer, and signed-deal measurement lives in HubSpot lifecycle imports.

Conversion-source check

  • Live Google Ads primary HubSpot lifecycle conversions are imported click conversions, not app/browser pixels: HubSpot - Lead (7210738292), HubSpot - Marketing Qualified Lead (7210866133), and HubSpot - Opportunity (7210867384) all have type UPLOAD_CLICKS, include_in_conversions_metric=true, and primary_for_goal=true.
  • Purchases/signed deals are treated as HubSpot lifecycle state, not product-app ad pixels. HubSpot - Customer (7210867654) is the CRM-side customer/purchase-class import; it is intentionally secondary today.
  • Purchase (2) (7524378447) is a legacy web conversion action, but it is not primary, not included in Conversions, and has 0 all-time conversions in the account query. Removing product-app ad pixels does not remove a meaningful biddable app conversion path.
  • Book a Call (7215323435) remains a primary web conversion from the marketing site /demo, not from the product app. So the practical biddable source mix is HubSpot imports plus marketing-site demo booking, not product-app pixels.

Portal check

  • Checked apps/portal for GTM, Google Ads/GA4, LinkedIn Insight/CAPI, dataLayer, lintrk, and marketing-pixel envs.
  • No portal code changes needed: portal is already PostHog-only via AnalyticsProvider and NEXT_PUBLIC_POSTHOG_* envs.

Validation

  • Targeted ESLint passed.
  • Targeted Prettier passed for changed TS/TSX/README files.
  • git diff --check and git diff --cached --check passed.
  • No product-app marketing tracking symbols found by rg.
  • No portal marketing tracking symbols found by rg.
  • Full bun --filter @trycompai/app typecheck still fails on existing generated DB/test/auth typing issues unrelated to this patch.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026

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

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Apr 30, 2026 1:43am
comp-framework-editor Ready Ready Preview, Comment Apr 30, 2026 1:43am
portal Ready Ready Preview, Comment Apr 30, 2026 1:43am

Request Review

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 30, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
CompAI 🟢 Ready View Preview Apr 30, 2026, 1:18 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@claudfuen claudfuen force-pushed the codex/app-marketing-pixels-gate branch from d1fec16 to c56a1fb Compare April 30, 2026 01:39
@claudfuen claudfuen marked this pull request as ready for review April 30, 2026 01:40
@claudfuen claudfuen merged commit 26d75be into main Apr 30, 2026
13 of 15 checks passed
@claudfuen claudfuen deleted the codex/app-marketing-pixels-gate branch April 30, 2026 01:41
claudfuen pushed a commit that referenced this pull request Apr 30, 2026
# [3.39.0](v3.38.1...v3.39.0) (2026-04-30)

### Bug Fixes

* **api:** correct fileData max-length to 100MB and allow localhost in URL validation ([6d56a2d](6d56a2d))
* **api:** correct type casting for rawBody in main.ts ([7e624d4](7e624d4))
* **app:** remove product app marketing pixels ([#2716](#2716)) ([26d75be](26d75be))
* **background-checks:** fix 13 bugs across billing, webhooks, custom uploads, and UI ([d5df5db](d5df5db))
* **background-checks:** remove employee PII from sessionStorage ([ea082b3](ea082b3)), closes [#133](#133)
* **background-checks:** security hardening across payment flow, validation, and logging ([8b3b39b](8b3b39b))
* **pentest:** bump split-view breakpoint from md to xl for tablet support ([efd1c85](efd1c85))

### Features

* **background-checks:** add employee background checks ([0456df6](0456df6))
@claudfuen
Copy link
Copy Markdown
Contributor Author

🎉 This PR is included in version 3.39.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant