refactor(experiments): drop conversions chart mode toggle

[Zaimwa9]

Thanks for submitting a PR! Please check the boxes below:

• I have read the Contributing Guide.
• I have added information to docs/ if required so people know about the feature.
• I have filled in the "Changes" section below.
• I have filled in the "How did you test this code" section below.

Changes

Removes the cumulative/daily toggle from the experiment "Conversions over time" chart — the chart is now always cumulative.

• Drops InlinePillToggle and its mode state from ExperimentConversionRateCard.
• Drops ConversionMode and the mode argument from buildConversionChartData; AccumulatedBucket no longer tracks per-bucket increments.
• Series labels are always "<variant> converted".

How did you test this code?

Updated deriveConversionRate unit tests (daily case removed); results/__tests__ passes, 85 tests. Checked the card renders the cumulative series with the toggle gone.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
flagsmith-frontend-preview	Ready	Preview	Sep 8, 2026 1:59pm UTC
flagsmith-frontend-staging	Ready	Preview	Sep 8, 2026 1:59pm UTC

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Ignored	Preview	Sep 8, 2026 1:59pm UTC

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 4d8df846-7c3a-4b16-94d5-dd1821292a7d

📥 Commits

Reviewing files that changed from the base of the PR and between e5a1ec0 and b4969aa.

📒 Files selected for processing (3)

• frontend/web/components/experiments/results/ExperimentConversionRateCard/ExperimentConversionRateCard.tsx
• frontend/web/components/experiments/results/__tests__/deriveConversionRate.test.ts
• frontend/web/components/experiments/results/deriveConversionRate.ts

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.

---

📝 Walkthrough

Walkthrough

The conversion chart now produces cumulative converted totals only. The daily conversion mode, per-bucket increment data, mode state, and cumulative/daily toggle were removed. Chart construction no longer accepts a mode argument. Tests now use the default cumulative behaviour and no longer cover daily series output.

Priority: ⬇️ Low — Defer this narrow experiment chart refactor because it only removes the daily conversion toggle and keeps cumulative data display.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to b4969

The conversions chart now always displays cumulative converted totals, with the daily toggle and daily-series behavior removed. The implementation and tests are aligned with this behavior.

Warning

Git: CodeRabbit could not clone the repository, so clone-backed analysis was skipped and this review may be incomplete. Verify repository clone access, such as SSH credentials, before requesting another full review. If clone access is intentionally unavailable, use path_filters to narrow the review scope.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

Docker builds report

Image	Build Status	Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-8484	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-8484	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-api:pr-8484	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith:pr-8484	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-8484	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-frontend:pr-8484	Finished ✅	Results ✅

[github-actions]

✅ private-cloud · depot-ubuntu-latest-16 — run #20293 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

 3 passed

Details

 3 tests across 3 suites
 32.6 seconds
 b4969aa
 🔄 Run: #20293 (attempt 1)

🗂️ Previous results

✅ private-cloud · depot-ubuntu-latest-arm-16 — run #20293 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

 1 passed

Details

 1 test across 1 suite
 1 minute, 17 seconds
 b4969aa
 🔄 Run: #20293 (attempt 1)

✅ oss · depot-ubuntu-latest-16 — run #20293 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

 1 passed

Details

 1 test across 1 suite
 40.3 seconds
 b4969aa
 🔄 Run: #20293 (attempt 1)

✅ oss · depot-ubuntu-latest-arm-16 — run #20293 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

 1 passed

Details

 1 test across 1 suite
 46.1 seconds
 b4969aa
 🔄 Run: #20293 (attempt 1)

[github-actions]

Visual Regression

19 screenshots compared. See report for details.
View full report