Skip to content
This repository has been archived by the owner on Apr 10, 2023. It is now read-only.

chore: show help text when creating rule w no variants #22

Merged
merged 3 commits into from
Jan 13, 2023

Conversation

markphelps
Copy link
Contributor

Fixes: FLI-152

We do allow creating rules with no variants, they simply return match: true with value: empty at evaluation time if it matches a segment.

This PR shows more helpful state/message instead of showing a variant select box with no options.

It also adds validation on the segmentKey at creation time

CleanShot 2023-01-12 at 16 43 55@2x

TODO: we still need to add validation of rule editing (for multi-variants) on segmentKey

Copy link
Contributor

@darinmclain darinmclain left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from fix-useeffect-deps to main January 13, 2023 12:18
@markphelps markphelps enabled auto-merge (squash) January 13, 2023 12:32
@markphelps markphelps merged commit be59ba4 into main Jan 13, 2023
@markphelps markphelps deleted the show-help-for-no-variants branch January 13, 2023 12:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants