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

Enable whitespace selection in choices #175

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

kyleperik
Copy link
Contributor

A string type with a choice that contains whitespace on the outer part cannot be selected, or it will generate an error. See the modified test for an example.

To fix this, I utilize the stripped/trimmed version of the string value only for validating whether the field is empty, otherwise the raw value is used.

Copy link

@NathanBaldwin NathanBaldwin left a comment

Choose a reason for hiding this comment

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

Good find! Thanks!

@kyleperik
Copy link
Contributor Author

Here's a link to when this was introduced, to me this indicates the empty validation was all this was needed for.

#60

@bhch bhch merged commit 3892068 into bhch:master Jul 18, 2024
17 checks passed
@bhch
Copy link
Owner

bhch commented Jul 18, 2024

Thank you for this fix. Will release soon.

@bhch
Copy link
Owner

bhch commented Sep 18, 2024

Released in v2.23.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants