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
talissoncosta added a commit that referenced this pull request Aug 13, 2026
Notify me and the feedback link fired onboarding funnel events carrying
environment, organisation and project ids. That records that some
organisation wants this and leaves nobody to reply to, so "we'll be in
touch" was a promise we could not keep.

They now also report where the other fake doors report, with the email
and organisation name, matching the segment sources door in #8272. The
funnel events stay as they were, since they are steps rather than signal.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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