Skip to content

Conversation

@rafaeelaudibert
Copy link
Member

@rafaeelaudibert rafaeelaudibert commented Dec 5, 2025

I sometimes wanna learn what percentage a flag is at for each variant at a glance, but it's impossible to do that without opening the flag right now. Let's change that! Let's add small blobs on the Release conditions screen with the percentages we calculated

image.png

@rafaeelaudibert rafaeelaudibert requested review from a team and MattBro and removed request for a team December 5, 2025 22:23
Copy link
Member Author

rafaeelaudibert commented Dec 5, 2025

@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2025

Size Change: +44 B (0%)

Total Size: 3.51 MB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 3.51 MB +44 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.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@posthog-bot
Copy link
Contributor

Visual regression: Storybook UI snapshots updated

Changes: 2 snapshots (2 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 →

@rafaeelaudibert rafaeelaudibert force-pushed the rafa/sync-feature-flags-handle-multivariate branch from 4457ca0 to fd072a6 Compare December 6, 2025 01:39
@rafaeelaudibert rafaeelaudibert force-pushed the rafa/display-multivariate-percentages-on-index-screen branch from f667413 to d7255ca Compare December 6, 2025 01:39
@rafaeelaudibert rafaeelaudibert requested review from a team and MattBro and removed request for a team December 6, 2025 01:40
@rafaeelaudibert rafaeelaudibert force-pushed the rafa/display-multivariate-percentages-on-index-screen branch from d651b05 to f1a876a Compare December 6, 2025 01:56
@posthog-bot
Copy link
Contributor

Visual regression: Storybook UI snapshots updated

Changes: 2 snapshots (2 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 →

@github-project-automation github-project-automation bot moved this from In Review to Approved in Feature Flags Dec 8, 2025
@rafaeelaudibert rafaeelaudibert changed the base branch from rafa/sync-feature-flags-handle-multivariate to graphite-base/42878 December 8, 2025 20:31
rafaeelaudibert and others added 5 commits December 8, 2025 20:32
I sometimes wanna learn what percentage a flag is at for each variant at a glance, but it's impossible to do that without opening the flag right now. Let's change that! Let's add small blobs on the `Release conditions` screen with the percentages we calculated
That looked weird and ugly
DIsplaying it with a compact visual looks better on the FF index page
@rafaeelaudibert rafaeelaudibert force-pushed the rafa/display-multivariate-percentages-on-index-screen branch from d10d905 to aa15ab0 Compare December 8, 2025 20:32
@posthog-project-board-bot posthog-project-board-bot bot moved this from Approved to In Review in Feature Flags Dec 8, 2025
@graphite-app graphite-app bot changed the base branch from graphite-base/42878 to master December 8, 2025 20:32
@rafaeelaudibert rafaeelaudibert force-pushed the rafa/display-multivariate-percentages-on-index-screen branch from aa15ab0 to 30c5a9c Compare December 8, 2025 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

4 participants