Skip to content

Conversation

@andrewjmcgehee
Copy link
Contributor

Problem

CustomerIO is a high value source for us interally. Add it as unreleased.

Changes

Add unreleased source.

How did you test this code?

Ran locally to verify source shows up as unreleased.

Changelog: (features only) Is this feature complete?

No

@posthog-bot posthog-bot requested a review from a team November 1, 2025 18:56
@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

Size Change: +13 B (0%)

Total Size: 3.34 MB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 3.34 MB +13 B (0%)

compressed-size-action

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

10 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

@posthog-bot
Copy link
Contributor

📸 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 14)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 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 14)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

⚠️ Flapping Screenshots Detected

The following screenshots have been updated multiple times by the bot, indicating test instability:

  • playwright/__snapshots__/pageview-trends-insight.png

What this means:

  • These screenshots are changing on each test run (timing issues, rendering differences, etc.)
  • This prevents reliable verification and blocks auto-merge
  • Human intervention is required

How to fix:

  1. Investigate flakiness: Check test timing, wait for elements, stabilize animations
  2. Fix the underlying issue: Don't just update snapshots repeatedly
  3. Verify locally: Run tests multiple times to ensure stability
  4. Push your fix: This will reset the flap counter

If you need to proceed anyway:

  • Fix the test flakiness first (recommended)
  • Or manually verify snapshots are acceptable and revert the bot commits, then push a fix

The workflow has been failed to prevent merging unstable tests.

@posthog-bot
Copy link
Contributor

📸 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 14)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

⚠️ Flapping Screenshots Detected

The following screenshots have been updated multiple times by the bot, indicating test instability:

  • playwright/__snapshots__/pageview-trends-insight.png

What this means:

  • These screenshots are changing on each test run (timing issues, rendering differences, etc.)
  • This prevents reliable verification and blocks auto-merge
  • Human intervention is required

How to fix:

  1. Investigate flakiness: Check test timing, wait for elements, stabilize animations
  2. Fix the underlying issue: Don't just update snapshots repeatedly
  3. Verify locally: Run tests multiple times to ensure stability
  4. Push your fix: This will reset the flap counter

If you need to proceed anyway:

  • Fix the test flakiness first (recommended)
  • Or manually verify snapshots are acceptable and revert the bot commits, then push a fix

The workflow has been failed to prevent merging unstable tests.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

⚠️ Flapping Screenshots Detected

The following screenshots have been updated multiple times by the bot, indicating test instability:

  • playwright/__snapshots__/pageview-trends-insight.png

What this means:

  • These screenshots are changing on each test run (timing issues, rendering differences, etc.)
  • This prevents reliable verification and blocks auto-merge
  • Human intervention is required

How to fix:

  1. Investigate flakiness: Check test timing, wait for elements, stabilize animations
  2. Fix the underlying issue: Don't just update snapshots repeatedly
  3. Verify locally: Run tests multiple times to ensure stability
  4. Push your fix: This will reset the flap counter

If you need to proceed anyway:

  • Fix the test flakiness first (recommended)
  • Or manually verify snapshots are acceptable and revert the bot commits, then push a fix

The workflow has been failed to prevent merging unstable tests.

@posthog-bot
Copy link
Contributor

📸 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 15)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

⚠️ Flapping Screenshots Detected

The following screenshots have been updated multiple times by the bot, indicating test instability:

  • playwright/__snapshots__/pageview-trends-insight.png

What this means:

  • These screenshots are changing on each test run (timing issues, rendering differences, etc.)
  • This prevents reliable verification and blocks auto-merge
  • Human intervention is required

How to fix:

  1. Investigate flakiness: Check test timing, wait for elements, stabilize animations
  2. Fix the underlying issue: Don't just update snapshots repeatedly
  3. Verify locally: Run tests multiple times to ensure stability
  4. Push your fix: This will reset the flap counter

If you need to proceed anyway:

  • Fix the test flakiness first (recommended)
  • Or manually verify snapshots are acceptable and revert the bot commits, then push a fix

The workflow has been failed to prevent merging unstable tests.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 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 11)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

⚠️ Flapping Screenshots Detected

The following screenshots have been updated multiple times by the bot, indicating test instability:

  • playwright/__snapshots__/pageview-trends-insight.png

What this means:

  • These screenshots are changing on each test run (timing issues, rendering differences, etc.)
  • This prevents reliable verification and blocks auto-merge
  • Human intervention is required

How to fix:

  1. Investigate flakiness: Check test timing, wait for elements, stabilize animations
  2. Fix the underlying issue: Don't just update snapshots repeatedly
  3. Verify locally: Run tests multiple times to ensure stability
  4. Push your fix: This will reset the flap counter

If you need to proceed anyway:

  • Fix the test flakiness first (recommended)
  • Or manually verify snapshots are acceptable and revert the bot commits, then push a fix

The workflow has been failed to prevent merging unstable tests.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

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/0897_alter_externaldatasource_source_type.py

BEGIN;
--
-- Alter field source_type on externaldatasource
--
-- (no-op)
COMMIT;
@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

🔍 Migration Risk Analysis

We've analyzed your migrations for potential risks.

Summary: 0 Safe | 1 Needs Review | 0 Blocked

⚠️ Needs Review

May have performance impact

posthog.0897_alter_externaldatasource_source_type
  └─ #1 ⚠️ AlterField
     Field alteration may cause table locks or data loss (check if changing type or constraints)
     model: externaldatasource, field: source_type, field_type: CharField
@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

⚠️ Flapping Screenshots Detected

The following screenshots have been updated multiple times by the bot, indicating test instability:

  • playwright/__snapshots__/pageview-trends-insight.png

What this means:

  • These screenshots are changing on each test run (timing issues, rendering differences, etc.)
  • This prevents reliable verification and blocks auto-merge
  • Human intervention is required

How to fix:

  1. Investigate flakiness: Check test timing, wait for elements, stabilize animations
  2. Fix the underlying issue: Don't just update snapshots repeatedly
  3. Verify locally: Run tests multiple times to ensure stability
  4. Push your fix: This will reset the flap counter

If you need to proceed anyway:

  • Fix the test flakiness first (recommended)
  • Or manually verify snapshots are acceptable and revert the bot commits, then push a fix

The workflow has been failed to prevent merging unstable tests.

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

Labels

None yet

3 participants