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

Boolean tag data type can not change value #155

Closed
tedsch opened this issue Sep 25, 2024 · 1 comment · Fixed by #156
Closed

Boolean tag data type can not change value #155

tedsch opened this issue Sep 25, 2024 · 1 comment · Fixed by #156

Comments

@tedsch
Copy link

tedsch commented Sep 25, 2024

With V2.0 and Ignition 8.1 can not change the value of a boolean tag. Setup a string, int and boolean memory tag on a Flexy unit. Was able to change the value of the string and int tags but not the boolean tag.

V1.2 and Ignition 8.1 you can change the value of a boolean data type tag.

@it-hms
Copy link
Contributor

it-hms commented Sep 26, 2024

I think we understand why this is happening.

The following will append true or false for Boolean objects, but the Flexy UpdateTagForm handles 1 or 0.

When we make the next release, we will pull in this issue.

it-hms added a commit that referenced this issue Sep 27, 2024
The Flexy UpdateTagForm request requires that boolean tag values are 1
or 0 and not true or false. This change updates the
EwonUpdateTagValuesRequest class to convert boolean tag values to 1 or
0.
Closes #155
@it-hms it-hms mentioned this issue Sep 30, 2024
it-hms added a commit that referenced this issue Sep 30, 2024
The Flexy UpdateTagForm request requires that boolean tag values are 1
or 0 and not true or false. This change updates the
EwonUpdateTagValuesRequest class to convert boolean tag values to 1 or
0.
Closes #155
@it-hms it-hms closed this as completed in d177f7d Oct 8, 2024
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 a pull request may close this issue.

2 participants