Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support OR-ing multiple sets of segment conditions together #3947

Open
rolodato opened this issue May 14, 2024 · 3 comments
Open

Support OR-ing multiple sets of segment conditions together #3947

rolodato opened this issue May 14, 2024 · 3 comments
Labels
front-end Issue related to the React Front End Dashboard improvement

Comments

@rolodato
Copy link
Contributor

Is your feature request related to a problem? Please describe.

When defining a segment, we can only define conditions in the shape (A or B or ...) AND/AND NOT (D or E or ...) AND/AND NOT .... It's not possible to OR multiple sets of conditions together.

Describe the solution you'd like.

Support OR-ing multiple sets of conditions together when defining a segment. Unsure what the UI would look like for this yet.

Describe alternatives you've considered

Implementing the logic in the application (i.e. using Flagsmith as a remote config store and not for its evaluation engine)

Additional context

No response

@matthewelwell matthewelwell added improvement front-end Issue related to the React Front End Dashboard labels May 15, 2024
@matthewelwell
Copy link
Contributor

@kyle-ssg I believe the BE can already support this, but I'm not sure how easy it will be to represent on the FE.

@kyle-ssg
Copy link
Member

kyle-ssg commented May 17, 2024

Im really unsure as to what the UI would look like, it might be worth getting Akveo to mock a design. I haven't seen this attempted before in products I don't think.

I feel like this could lead to people getting very confused if it's not done right.

@rolodato
Copy link
Contributor Author

I can't think of anything feasible besides a tree-style editor to build arbitrarily complex boolean conditions, similar to https://querybuilder.js.org/ but with an actually nice design :)

This will likely require extracting "New Segment" from the current modal into its own page, to get a reasonable amount of screen space to work with.

If we do allow customers to define more complex conditions, IMO we should provide some "tester" functionality that lets customers play around with sample values to sanity-check their segment definitions and explain what is causing a certain result. This could be part of the segment editor or its own page (separate page would be my preference), which could help explain if segment or identity overrides are being applied and not need to create fake identities just for testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front-end Issue related to the React Front End Dashboard improvement
Projects
None yet
Development

No branches or pull requests

3 participants