Skip to content

Conversation

@kyle-ssg
Copy link
Member

@kyle-ssg kyle-ssg commented Dec 2, 2025

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

  • 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?
  • I have used a Conventional Commit title for this Pull Request

Changes

  • Migrates feature settings, feature value and create feature to typescript components
  • Migrates Row / FormGroup to fix imports
  • Simplifies interface around the create feature modal, this will eventually mean the RTK/redux store no longer needs to create new objects
    • Editing project/feature related stuff is adjusting the ProjectFlag with a single onChange
    • Editing the remote config, enabled state and MV percentages is adjusting a FeatureState with a single on Chnange
  • Minor: add tag button same height as tags
image

How did you test this code?

E2E covers regression testing on everything this affects

kyle-ssg and others added 30 commits August 5, 2025 16:27
Co-authored-by: Zaimwa9 <wadii.zaim@flagsmith.com>
# Conflicts:
#	frontend/web/components/pages/UserPage.tsx
Copy link
Contributor

@talissoncosta talissoncosta left a comment

Choose a reason for hiding this comment

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

I’ve noticed some unusual behavior during the QA session and would like to clarify it before moving forward.

cursor[bot]

This comment was marked as outdated.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@talissoncosta
Copy link
Contributor

V1 not requesting CR to update segment overrides

https://www.loom.com/share/e75ccdbd4dc044ec94942da76daa2a32

@kyle-ssg kyle-ssg merged commit 00085cd into main Jan 20, 2026
27 checks passed
@kyle-ssg kyle-ssg deleted the chore/create-flag-migration branch January 20, 2026 11:59
talissoncosta added a commit that referenced this pull request Jan 23, 2026
… override

The 'Enabled by default' toggle and its tooltip were incorrectly shown
when editing an identity override. This was a regression introduced in
PR #6356 where the CreateFeature component was extracted with isEdit
hardcoded to false.

When editing an identity override, isEdit should be true so that:
- The toggle shows 'Enabled' instead of 'Enabled by default'
- The tooltip about initial enabled state is not shown

Fixes #6580

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Issue related to the REST API chore front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants