posthog icon indicating copy to clipboard operation
posthog copied to clipboard

feat: Allow to turn test events off for Revenue Analytics

Open rafaeelaudibert opened this issue 7 months ago â€ĸ 6 comments

  • Introduced filterTestAccounts boolean in schema.json and updated required fields.
  • Added FilterTestAccountsConfiguration component for UI settings.
  • Integrated filter logic in revenue analytics views to exclude test accounts from analytics.
  • Updated relevant models and migrations to support the new feature.
image

rafaeelaudibert avatar Jun 13 '25 21:06 rafaeelaudibert

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 13 '25 21:06 posthog-bot

📸 UI snapshots have been updated

18 snapshot changes in total. 0 added, 18 modified, 0 deleted:

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 13 '25 21:06 posthog-bot

Size Change: 0 B

Total Size: 2.57 MB

â„šī¸ View Unchanged
Filename Size
frontend/dist/toolbar.js 2.57 MB

compressed-size-action

github-actions[bot] avatar Jun 14 '25 17:06 github-actions[bot]

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 15 '25 03:06 posthog-bot

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

[!CAUTION]

Detected flapping snapshots

These snapshots have auto-updated more than once since the last human commit:

  • scenes-app-insights-funnels--funnel-top-to-bottom-breakdown--dark.png (chromium, shard 2)

The flippy-flappies are deadly and must be fixed ASAP. They're productivity killers. Run pnpm storybook locally and make the fix now. (Often, the cause is ResizeObserver being used instead of the better CSS container queries.)

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 15 '25 03:06 posthog-bot

📸 UI snapshots have been updated

16 snapshot changes in total. 0 added, 16 modified, 0 deleted:

  • chromium: 0 added, 16 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 16 '25 16:06 posthog-bot

Migration SQL Changes

Hey 👋, we've detected some migrations on this PR. Here's the SQL output for each migration, make sure they make sense:

posthog/migrations/0771_teamrevenueanalyticsconfig_filter_test_accounts_and_more.py

BEGIN;
--
-- Add field filter_test_accounts to teamrevenueanalyticsconfig
--
ALTER TABLE "posthog_teamrevenueanalyticsconfig" ADD COLUMN "filter_test_accounts" boolean DEFAULT false NOT NULL;
ALTER TABLE "posthog_teamrevenueanalyticsconfig" ALTER COLUMN "filter_test_accounts" DROP DEFAULT;
--
-- Raw SQL operation
--
ALTER TABLE posthog_teamrevenueanalyticsconfig ALTER COLUMN filter_test_accounts SET DEFAULT false;
COMMIT;

github-actions[bot] avatar Jun 16 '25 17:06 github-actions[bot]