Skip to content

feat: use dropdown for values with context environment - #5832

Merged
Zaimwa9 merged 14 commits into
mainfrom
feat/use-dropdown-for-values-with-context-environment
Aug 11, 2025
Merged

feat: use dropdown for values with context environment#5832
Zaimwa9 merged 14 commits into
mainfrom
feat/use-dropdown-for-values-with-context-environment

Conversation

@Zaimwa9

@Zaimwa9 Zaimwa9 commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

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

When using the Environment name context value (and not PERCENTAGE_SPLIT), the input value becomes a dropdown with the project's environment values

How did you test this code?

N/A
image

image

https://www.loom.com/share/9c00c9440fed4a499ea16192a42cb932?focus_title=1&muted=1&from_recorder=1

@Zaimwa9
Zaimwa9 requested a review from a team as a code owner July 25, 2025 08:37
@Zaimwa9
Zaimwa9 requested review from tiagoapolo and removed request for a team July 25, 2025 08:37
@vercel

vercel Bot commented Jul 25, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
flagsmith-frontend-preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 11, 2025 9:55am
flagsmith-frontend-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 11, 2025 9:55am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview Aug 11, 2025 9:55am

@github-actions github-actions Bot added the front-end Issue related to the React Front End Dashboard label Jul 25, 2025
@github-actions

github-actions Bot commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

Docker builds report

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

@khvn26 khvn26 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I see the rationale behind using a dropdown. Consider, however, a case when an environment used in the segment condition was deleted. I believe we should use a searchable dropdown allowing arbitrary values — should work nicely with a warning added by #5805.

@khvn26

khvn26 commented Jul 25, 2025

Copy link
Copy Markdown
Member

Closes #5804.

@github-actions github-actions Bot removed the feature New feature or request label Jul 25, 2025
Co-authored-by: Tiago Paiva <tiago@tiagopaiva.me>
tiagoapolo
tiagoapolo previously approved these changes Aug 5, 2025
Zaimwa9 added 3 commits August 7, 2025 09:58
…of github.com:Flagsmith/flagsmith into feat/use-dropdown-for-values-with-context-environment
@github-actions github-actions Bot added feature New feature or request and removed feature New feature or request labels Aug 7, 2025
tiagoapolo
tiagoapolo previously approved these changes Aug 8, 2025
@Zaimwa9
Zaimwa9 requested a review from kyle-ssg August 11, 2025 07:36
@Zaimwa9
Zaimwa9 removed the request for review from kyle-ssg August 11, 2025 12:25
@Zaimwa9
Zaimwa9 merged commit 6153206 into main Aug 11, 2025
25 checks passed
@Zaimwa9
Zaimwa9 deleted the feat/use-dropdown-for-values-with-context-environment branch August 11, 2025 12:25
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.

4 participants