fix: OR button hiding and empty condtions

[MatheusLasserre]

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

• I have run pre-commit to check linting
• 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

• I've changed how the last index of the Condition Row is calculated, so it doesn't count the deleted ones, making the OR button show on the last visible row.
• I've changed the setRuleProperty function so it sets a default deletion value and property: "deleted", as @kyle-ssg suggested.

How did you test this code?

I've created all conditions types inside the segment overrides, filled it with dummy values, deleted it all and then tried to save the new segment. The API didn't complain about the default deletion values, and the OR button was there all the time.

fixes #3244

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 11, 2024 9:43am
flagsmith-frontend-preview	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 11, 2024 9:43am
flagsmith-frontend-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 11, 2024 9:43am

[vercel]

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

A member of the Team first needs to authorize it.

[github-actions]

Uffizzi Preview deployment-44123 was deleted.

[kyle-ssg]

Awesome!