Skip to content

Conversation

Zaimwa9
Copy link
Contributor

@Zaimwa9 Zaimwa9 commented Jul 31, 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

Some operators include an appended part OPERATOR:{append} that need to be cleaned-up when setting their values
E.G GREATER_THAN:semver breaking updates

Regression from here

How did you test this code?

  • Manually
image

Fixed and correct condition payload example

{
    "id": 268,
    "operator": "GREATER_THAN_INCLUSIVE",
    "property": "$.environment.name",
    "value": "18.21.2:semver",
    "description": null,
    "version_of": null
}

@Zaimwa9 Zaimwa9 requested a review from a team as a code owner July 31, 2025 12:19
@Zaimwa9 Zaimwa9 requested review from tiagoapolo and removed request for a team July 31, 2025 12:19
Copy link

vercel bot commented Jul 31, 2025

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 Jul 31, 2025 0:19am
flagsmith-frontend-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2025 0:19am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Jul 31, 2025 0:19am

Copy link
Contributor

github-actions bot commented Jul 31, 2025

Docker builds report

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

@Zaimwa9 Zaimwa9 merged commit df27bb1 into main Jul 31, 2025
27 checks passed
@Zaimwa9 Zaimwa9 deleted the fix/split-appended-part-of-the-operator branch July 31, 2025 13:31
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