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.

Sign up for GitHub

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

Jump to bottom

[shelly] relay#autoOn and relay#autoOff don't accept UBM #8431

Closed
markus7017 opened this issue Sep 8, 2020 · 1 comment · Fixed by #8396
Closed

[shelly] relay#autoOn and relay#autoOff don't accept UBM #8431

markus7017 opened this issue Sep 8, 2020 · 1 comment · Fixed by #8396
Assignees
@markus7017
Labels
bug An unexpected problem or unintended behavior of an add-on work in progress A PR that is not yet ready to be merged
Milestone
2.5.9

Comments

@markus7017
Copy link
Contributor

The item type for channel autoOn, autoOff is now Number:Time. The current value is reported as "xx s", which is correct. However, when sending 30s to those channels the value doesn't change. In fact the DEBUG log shows a format exception.

Handling of QuantityType has been added, so both Number and Number:Time are accepted now.
@markus7017 markus7017 added bug An unexpected problem or unintended behavior of an add-on work in progress A PR that is not yet ready to be merged labels Sep 8, 2020
@markus7017 markus7017 added this to the 2.5.9 milestone Sep 8, 2020
@markus7017 markus7017 self-assigned this Sep 8, 2020
@markus7017 markus7017 linked a pull request Sep 8, 2020 that will close this issue
[shelly] Add channels device#deviceName, relay#outputName + refactoring #8396
Merged
markus7017 added a commit to markus7017/openhab-addons that referenced this issue Sep 8, 2020
@markus7017
Fix for openhab#8431: Handling of UBM for channels autoOn and autoOff
1f5ef34 
Signed-off-by: Markus Michels <markus7017@gmail.com>
@markus7017
Copy link
Contributor Author

@fwolter Thanks for you support on reviewing this PR 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markus7017 markus7017

Labels
bug An unexpected problem or unintended behavior of an add-on work in progress A PR that is not yet ready to be merged
Projects
None yet
Milestone
2.5.9
Development

Successfully merging a pull request may close this issue.

[shelly] Add channels device#deviceName, relay#outputName + refactoring markus7017/openhab-addons
1 participant
@markus7017