Skip to content

fix: remove v0.1 experiment flag toggle from create feature modal#7907

Open
Zaimwa9 wants to merge 1 commit into
mainfrom
fix/remove-experiment-flag-toggle
Open

fix: remove v0.1 experiment flag toggle from create feature modal#7907
Zaimwa9 wants to merge 1 commit into
mainfrom
fix/remove-experiment-flag-toggle

Conversation

@Zaimwa9

@Zaimwa9 Zaimwa9 commented Jun 29, 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

Removes the legacy "Experiment flag" toggle from the create/edit feature modal — this was a v0.1 artifact superseded by the experiment freeze mechanism which checks for running experiments via the API.

How did you test this code?

  • Manually

@Zaimwa9 Zaimwa9 requested a review from a team as a code owner June 29, 2026 15:45
@Zaimwa9 Zaimwa9 requested review from talissoncosta and removed request for a team June 29, 2026 15:45
@vercel

vercel Bot commented Jun 29, 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, Comment Jun 29, 2026 3:47pm
flagsmith-frontend-staging Ready Ready Preview, Comment Jun 29, 2026 3:47pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Jun 29, 2026 3:47pm

Request Review

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Docker builds report

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

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

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

passed  4 passed

Details

stats  4 tests across 3 suites
duration  19.5 seconds
commit  a63233b
info  🔄 Run: #17924 (attempt 1)

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

passed  4 passed

Details

stats  4 tests across 3 suites
duration  19.2 seconds
commit  a63233b
info  🔄 Run: #17924 (attempt 1)

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

passed  1 passed

Details

stats  1 test across 1 suite
duration  1 minute, 11 seconds
commit  a63233b
info  🔄 Run: #17924 (attempt 1)

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

passed  6 passed

Details

stats  6 tests across 5 suites
duration  56.3 seconds
commit  a63233b
info  🔄 Run: #17924 (attempt 1)

@github-actions

Copy link
Copy Markdown
Contributor

Visual Regression

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

import ConfirmToggleFeature from 'components/modals/ConfirmToggleFeature'
import ConfirmRemoveFeature from 'components/modals/ConfirmRemoveFeature'
import CreateFlagModal from 'components/modals/create-feature'
import CreateExperimentModal from 'components/modals/create-experiment'

@talissoncosta talissoncosta Jun 30, 2026

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.

Are we still gonna keep CreateExperimentModal ?
Just checking to prevent dead code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants