Skip to content

feat: restrict PostHog to SaaS builds and migrate Hotjar attrs to Pos…#1574

Merged
gugu merged 1 commit intomainfrom
posthog-saas-only
Feb 6, 2026
Merged

feat: restrict PostHog to SaaS builds and migrate Hotjar attrs to Pos…#1574
gugu merged 1 commit intomainfrom
posthog-saas-only

Conversation

@gugu
Copy link
Contributor

@gugu gugu commented Feb 6, 2026

…tHog

Gate PostHog initialization behind the saas environment flag so it is excluded from self-hosted builds. Replace all data-hj-suppress (Hotjar) attributes with data-ph-no-capture (PostHog) for session replay masking.

…tHog

Gate PostHog initialization behind the saas environment flag so it
is excluded from self-hosted builds. Replace all data-hj-suppress
(Hotjar) attributes with data-ph-no-capture (PostHog) for session
replay masking.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 6, 2026 12:11
@gugu gugu enabled auto-merge (squash) February 6, 2026 12:11
@gugu gugu merged commit 644bebb into main Feb 6, 2026
20 checks passed
@gugu gugu deleted the posthog-saas-only branch February 6, 2026 12:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR gates PostHog analytics to SaaS builds only and migrates privacy masking attributes from Hotjar to PostHog. The changes ensure that self-hosted builds exclude the PostHog analytics library while maintaining proper data privacy controls for session replays in SaaS builds.

Changes:

  • Wrapped PostHog initialization with environment.saas flag to exclude it from self-hosted builds
  • Replaced all data-hj-suppress attributes with data-ph-no-capture across 9 HTML template files to maintain session replay masking for PostHog

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
frontend/src/main.ts Added environment.saas conditional around PostHog initialization to restrict it to SaaS builds
frontend/src/app/components/ui-components/record-edit-fields/markdown/markdown.component.html Migrated Hotjar masking attribute to PostHog in markdown editor
frontend/src/app/components/ui-components/record-edit-fields/json-editor/json-editor.component.html Migrated Hotjar masking attribute to PostHog in JSON editor
frontend/src/app/components/ui-components/record-edit-fields/code/code.component.html Migrated Hotjar masking attribute to PostHog in code editor
frontend/src/app/components/dashboard/db-table-view/saved-filters-panel/saved-filters-panel.component.html Migrated Hotjar masking attribute to PostHog on filter chips
frontend/src/app/components/dashboard/db-table-view/db-table-view.component.html Migrated Hotjar masking attributes to PostHog on filter chips and table cells
frontend/src/app/components/dashboard/db-table-view/db-table-row-view/db-table-row-view.component.html Migrated Hotjar masking attribute to PostHog in row preview values
frontend/src/app/components/charts/chart-edit/chart-edit.component.html Migrated Hotjar masking attribute to PostHog in SQL query editor
frontend/src/app/components/audit/info-dialog/info-dialog.component.html Migrated Hotjar masking attributes to PostHog in audit log details

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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