Segment conditions are duplicated when saving multiple updates and not refreshing

[rolodato]

How are you running Flagsmith

• Self Hosted with Docker
• Self Hosted with Kubernetes
• SaaS at flagsmith.com
• Some other way (add details in description below)

Describe the bug

dupe.segment.mov

Steps To Reproduce

1. From an existing segment, add a condition A
2. Save changes, don't close the modal
3. Add another OR condition B
4. Save changes
5. Refresh the page - condition A is now duplicated

This happens because after the initial save, the frontend does not save the new ID that the API assigned to condition A. When saving condition B, it sends condition A without an ID, which the API interprets to be an entirely new condition.

First save - in this case, condition 1 existed in the segment before loading. Condition 2was added here.

Second save - condition 3 was added here, but condition 2 is sent again without an ID. This causes condition 2 to get duplicated.

Expected behavior

Don't duplicate conditions.

Screenshots

No response