posthog icon indicating copy to clipboard operation
posthog copied to clipboard

fix: push knowledge about the playlist down into the playlist component

Open pauldambra opened this issue 2 weeks ago â€ĸ 4 comments

splitting out safe steps from https://github.com/PostHog/posthog/pull/42358

push down as much knowledge about the playlist into the Playlist component as i can get away with

pauldambra avatar Dec 07 '25 12:12 pauldambra

  • #43314 Graphite
  • #42906 Graphite
  • #42902 Graphite 👈 (View in Graphite)
  • #42897 Graphite
  • #42854 Graphite: 1 other dependent PR (#42871 Graphite)
  • #42836 Graphite
  • #42831 Graphite
  • #42821 Graphite
  • #42819 Graphite
  • #42815 Graphite
  • master

This stack of pull requests is managed by Graphite. Learn more about stacking.

pauldambra avatar Dec 07 '25 12:12 pauldambra

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Size Change: 0 B

Total Size: 3.7 MB

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

compressed-size-action

github-actions[bot] avatar Dec 07 '25 12:12 github-actions[bot]

Visual regression: Storybook UI snapshots updated

Changes: 6 snapshots (6 modified, 0 added, 0 deleted)

What this means:

  • Snapshots have been automatically updated to match current rendering
  • Next CI run will switch to CHECK mode to verify stability
  • If snapshots change again, CHECK mode will fail (indicates flapping)

Next steps:

  • Review the changes to ensure they're intentional
  • Approve if changes match your expectations
  • If unexpected, investigate component rendering

Review snapshot changes →

posthog-bot avatar Dec 07 '25 12:12 posthog-bot

Visual regression: Storybook UI snapshots updated

Changes: 12 snapshots (0 modified, 12 added, 0 deleted)

What this means:

  • Snapshots have been automatically updated to match current rendering
  • Next CI run will switch to CHECK mode to verify stability
  • If snapshots change again, CHECK mode will fail (indicates flapping)

Next steps:

  • Review the changes to ensure they're intentional
  • Approve if changes match your expectations
  • If unexpected, investigate component rendering

Review snapshot changes →

posthog-bot avatar Dec 11 '25 11:12 posthog-bot

Visual regression: Storybook UI snapshots updated

Changes: 10 snapshots (0 modified, 10 added, 0 deleted)

What this means:

  • Snapshots have been automatically updated to match current rendering
  • Next CI run will switch to CHECK mode to verify stability
  • If snapshots change again, CHECK mode will fail (indicates flapping)

Next steps:

  • Review the changes to ensure they're intentional
  • Approve if changes match your expectations
  • If unexpected, investigate component rendering

Review snapshot changes →

posthog-bot avatar Dec 11 '25 16:12 posthog-bot

Merge activity

  • Dec 12, 4:51 PM UTC: @pauldambra merged this pull request with Graphite.

pauldambra avatar Dec 12 '25 16:12 pauldambra