Skip to content

feat: create segment sources fake door modal - #8272

Merged
Zaimwa9 merged 1 commit into
mainfrom
feat/create-segment-sources-fake-door
Aug 13, 2026
Merged

feat: create segment sources fake door modal#8272
Zaimwa9 merged 1 commit into
mainfrom
feat/create-segment-sources-fake-door

Conversation

@Zaimwa9

@Zaimwa9 Zaimwa9 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor
  • 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

Adds a "Create Segment" choice modal offering segment sources, gated behind the create_segment_with_external_sources feature flag (flag off: the button opens the existing drawer directly, unchanged).

  • Manually — closes the modal and opens the existing rules drawer.
  • From a CSV list, Amplitude, Mixpanel, Adobe Journey Manager — fake doors carrying a Beta chip. Selecting one fires a segment_source_clicked SDK event (source, email, organisation metadata) and reveals a "Request access to the beta" button, which fires segment_source_beta_requested, disables itself, and thanks the user — same pattern as the Experiments fake door.

How did you test this code?

Manually, against staging:

  1. Enable create_segment_with_external_sources for your organisation.
  2. Segments → Create Segment: the choice modal opens; with the flag disabled the drawer opens directly.
  3. Click "Manually" → modal closes, existing New Segment drawer opens.
  4. Click each source → card highlights, segment_source_clicked fires with the right source.
  5. Click "Request access to the beta" → segment_source_beta_requested fires, button disables with a thank-you message; selecting another source re-enables it.

@Zaimwa9
Zaimwa9 requested a review from a team as a code owner August 12, 2026 14:28
@Zaimwa9
Zaimwa9 requested review from talissoncosta and removed request for a team August 12, 2026 14:28
@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
flagsmith-frontend-preview Ready Ready Preview Aug 13, 2026 7:47am
flagsmith-frontend-staging Ready Ready Preview Aug 13, 2026 7:47am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Aug 13, 2026 7:47am

Request Review

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f2fd52c4-25dd-4bf4-abaa-e3343198dcd1

📥 Commits

Reviewing files that changed from the base of the PR and between b8410be and 0aede04.

📒 Files selected for processing (1)
  • frontend/web/components/icons/Icon.tsx

📝 Walkthrough

Walkthrough

Adds CreateSegmentSourcesModal with manual, CSV, Amplitude, Mixpanel, and Adobe Journey Manager options. The modal tracks source selection and beta-access requests through Flagsmith and prevents duplicate requests. SegmentsPage opens the modal when the feature flag is enabled and retains the existing drawer flow otherwise. Adds a wide modal style with an 860px maximum width.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score: ⚪ Minimal · up to 0aede

This localized frontend change adds a feature-flagged segment-source choice modal while preserving the existing flow when the flag is off; no actionable merge-blocking risk remains beyond normal checks and review.


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

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

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Docker builds report

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 3


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 49e7a95b-1adf-464f-ad0e-9991a9dd5936

📥 Commits

Reviewing files that changed from the base of the PR and between 6c88198 and fe9baf6.

📒 Files selected for processing (4)
  • frontend/web/components/modals/CreateSegmentSourcesModal/CreateSegmentSourcesModal.tsx
  • frontend/web/components/modals/CreateSegmentSourcesModal/index.ts
  • frontend/web/components/pages/SegmentsPage.tsx
  • frontend/web/styles/project/_modals.scss

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor
✅ private-cloud · depot-ubuntu-latest-16 — run #19284 (attempt 1)

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  54.5 seconds
commit  0aede04
info  🔄 Run: #19284 (attempt 1)

🗂️ Previous results
✅ private-cloud · depot-ubuntu-latest-arm-16 — run #19284 (attempt 1)

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

passed  3 passed

Details

stats  3 tests across 3 suites
duration  59.8 seconds
commit  0aede04
info  🔄 Run: #19284 (attempt 1)

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  44.9 seconds
commit  0aede04
info  🔄 Run: #19284 (attempt 1)

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  32.7 seconds
commit  0aede04
info  🔄 Run: #19284 (attempt 1)

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

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

passed  3 passed

Details

stats  3 tests across 3 suites
duration  55.2 seconds
commit  b8410be
info  🔄 Run: #19283 (attempt 1)

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  45.2 seconds
commit  b8410be
info  🔄 Run: #19283 (attempt 1)

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

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

passed  23 passed
skipped  1 skipped

Details

stats  24 tests across 18 suites
duration  1 minute, 21 seconds
commit  b8410be
info  🔄 Run: #19283 (attempt 1)

Skipped tests

firefox › tests/onboarding-tests.pw.ts › Onboarding › New user connects via the single-page onboarding flow @oss

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  41.2 seconds
commit  b8410be
info  🔄 Run: #19283 (attempt 1)

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  55.3 seconds
commit  ddd6fe9
info  🔄 Run: #19250 (attempt 1)

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  36.9 seconds
commit  ddd6fe9
info  🔄 Run: #19250 (attempt 1)

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  37.4 seconds
commit  ddd6fe9
info  🔄 Run: #19250 (attempt 1)

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

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  36.6 seconds
commit  ddd6fe9
info  🔄 Run: #19250 (attempt 1)

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Visual Regression

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 3


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 9fb342c0-98b5-4eac-b26f-fb17ad913749

📥 Commits

Reviewing files that changed from the base of the PR and between fe9baf6 and ddd6fe9.

📒 Files selected for processing (3)
  • frontend/e2e/helpers/e2e-helpers.playwright.ts
  • frontend/web/components/modals/CreateSegmentSourcesModal/CreateSegmentSourcesModal.scss
  • frontend/web/components/modals/CreateSegmentSourcesModal/CreateSegmentSourcesModal.tsx

talissoncosta
talissoncosta previously approved these changes Aug 12, 2026

@talissoncosta talissoncosta left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks pretty good! Congrats! Just a few suggestions. I don't wanna block you. However it would be nice to keep it clean and reuse the existing components.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 7c36ae63-a84d-44b7-b124-c65e413ff82c

📥 Commits

Reviewing files that changed from the base of the PR and between ddd6fe9 and b8410be.

📒 Files selected for processing (3)
  • frontend/web/components/icons/Icon.tsx
  • frontend/web/components/modals/CreateSegmentSourcesModal/CreateSegmentSourcesModal.scss
  • frontend/web/components/modals/CreateSegmentSourcesModal/CreateSegmentSourcesModal.tsx

Comment thread frontend/web/components/icons/Icon.tsx
@github-actions github-actions Bot added feature New feature or request and removed feature New feature or request labels Aug 13, 2026
@Zaimwa9
Zaimwa9 merged commit dc99d65 into main Aug 13, 2026
30 checks passed
@Zaimwa9
Zaimwa9 deleted the feat/create-segment-sources-fake-door branch August 13, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants