Skip to content

feat(Flags): Add remove option to feature settings#7898

Draft
Dexter2099 wants to merge 1 commit into
Flagsmith:mainfrom
Dexter2099:codex/feature-settings-remove-option
Draft

feat(Flags): Add remove option to feature settings#7898
Dexter2099 wants to merge 1 commit into
Flagsmith:mainfrom
Dexter2099:codex/feature-settings-remove-option

Conversation

@Dexter2099

Copy link
Copy Markdown

Feature settings currently show archive but not removal, which makes archive look like the only way to retire a feature. This adds the existing remove flow to the settings panel so users can delete a feature without returning to the feature list row menu.

Changes

  • Adds a Remove Feature action to the feature settings panel.
  • Reuses the existing typed confirmation and remove-feature toast flow.
  • Keeps removal behind the existing delete-feature permission and permanent-tag protection.
  • Covers the remove-button disabled states with a focused unit test.

Closes #7896

Validation

  • npm run test:unit -- --testPathPatterns=FeatureSettingsTab.utils.test.ts
  • npx eslint --fix web/components/modals/create-feature/index.tsx web/components/modals/create-feature/tabs/FeatureSettingsTab.tsx web/components/modals/create-feature/tabs/FeatureSettingsTab.utils.ts web/components/modals/create-feature/tabs/__tests__/FeatureSettingsTab.utils.test.ts
  • npx tsc --noEmit --pretty false 2>&1 | Select-String -Pattern 'web/components/modals/create-feature/index.tsx|web/components/modals/create-feature/tabs/FeatureSettingsTab.tsx|web/components/modals/create-feature/tabs/FeatureSettingsTab.utils.ts|FeatureSettingsTab.utils.test.ts' returned no touched-file errors.
  • npm run typecheck is not clean on this checkout due existing unrelated errors, beginning with common/constants.ts(435,36) and common/hooks/useCollapsibleHeight.ts(36,15).

Review effort: 2/5

@vercel

vercel Bot commented Jun 27, 2026

Copy link
Copy Markdown

@Dexter2099 is attempting to deploy a commit to the Flagsmith Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the front-end Issue related to the React Front End Dashboard label Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature settings is missing a "remove" option

1 participant