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

feat(ui): introduce FieldType.Boolean - WF-165 #732

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

madeindjs
Copy link
Collaborator

This new field is just an alias for FieldType.Text but it will make useEvaluator parse the value and accept "yes" or True and return the boolean value.

This unlock the usage of boolean value in the state like:

wf.init_state({
  "string": "yes",
  "true": True,
})

This is also the first step to introduce a new field setting UX for boolean value (like a checkbox or something).

@madeindjs madeindjs self-assigned this Jan 28, 2025
@madeindjs madeindjs changed the title feat(ui): introduce FieldType.Boolean - WF feat(ui): introduce FieldType.Boolean - WF-165 Jan 28, 2025
@madeindjs madeindjs force-pushed the WF-165 branch 3 times, most recently from 444370a to 7449183 Compare January 29, 2025 16:01
@madeindjs madeindjs marked this pull request as ready for review January 31, 2025 15:16
This new field is just an alias for `FieldType.Text` but it will
make `useEvaluator` parse the value and accept `"yes"` or `True`
and return the boolean value.

This unlock the usage of boolean value in the state like:

```py
wf.init_state({
  "string": "yes",
  "true": True,
})
```

This is also the first step to introduce a new field setting UX for
boolean value (like a checkbox or something).
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.

1 participant