Skip to content

[Marketing] Track OpenAI Ads Registrations - #398

Merged
gregagi merged 2 commits into
mainfrom
rasul/openai-ads-registration-pixel
Aug 4, 2026
Merged

[Marketing] Track OpenAI Ads Registrations#398
gregagi merged 2 commits into
mainfrom
rasul/openai-ads-registration-pixel

Conversation

@rasulkireev

Copy link
Copy Markdown
Contributor

Summary

Rowset can now attribute completed account registrations to OpenAI Ads without measuring visitors before they grant analytics consent. The sitewide pixel is enabled only when a deployment supplies a Pixel ID, and the conversion survives signup session rotation and navigation while consent is pending.

The implementation deliberately sends no email, user identifier, or other matching data. It also keeps self-hosted deployments disconnected by default and uses OpenAI's production-safe debug behavior.

Rollout

  • Set OPENAI_ADS_PIXEL_ID=8dYXMBukBoC779PidZY6ZX in the production environment.
  • Complete one controlled ad-click signup after deployment and confirm one registration_completed event in Ads Manager diagnostics.
  • Unset the environment variable to disable the integration without a code rollback.

Validation

  • make ci-local: lint, production asset build, migration check, Django checks, 61 frontend tests, and 1,282 backend tests passed.
  • OpenAI Ads setup verification passed with the Pixel requirement enabled.
  • CAPI secret exposure verification reported zero findings.

Compound Engineering

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Gate Summary

All enabled ReviewGate review angles passed.

Confidence Score: 5/5

Review angle Score Findings
General 5/5 0
Adversarial 5/5 0
Runtime phase Duration
Queue 636.00s
Startup 4.67s
Model review 34.25s
Publish 0.97s

Findings: 0 total, 0 blocking, 0 inline candidates
Notes: 3 note(s) in the JSON artifact.

No findings. Re-run ReviewGate if new commits land.

Important Files Changed

No files require special attention from this review.

Flowchart
flowchart TD
    A["Pull request update"] --> B["ReviewGate analyzes the latest commit"]
    B --> C["Model review stages"]
    C --> D["Structured JSON artifact"]
    D --> E["Confidence Score: 5/5"]
    D --> F["Canonical PR summary comment"]
    F --> G["Human or agent fixes findings"]
    E --> H["Status: passed"]
Loading

Reviews on this PR: 2 runs. Changed lines analyzed: 0. Total cost: $0.05. Latest commit analyzed: 84bfe0b4f4f97e2f0e19006ba309ec58d0fd4d10.

Comment thread apps/core/context_processors.py
@rasulkireev

Copy link
Copy Markdown
Contributor Author

ReviewGate agent disposition recorded for defect:apps.core.context.processors.py:openai.ads.session.pop.race.

@gregagi
gregagi merged commit 3dbf56d into main Aug 4, 2026
9 checks passed
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