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

changed Types #545

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

changed Types #545

wants to merge 1 commit into from

Conversation

DominikJaco
Copy link

tsoll,windowopenactivetime,boostactivetime changed to type mixed because logs showed "State value to set for "fritzdect.0.DECT_xxx.windowopenactivetime" has to be type "number" but received type "string"

tsoll,windowopenactivetime,boostactivetime changed to type mixed because logs showed  
"State value to set for "fritzdect.0.DECT_xxx.windowopenactivetime" has to be type "number" but received type "string"
@foxthefox
Copy link
Owner

I can not follow the issue itself.

windowopenactivetime is a number and default is 5 (min), and from my perspective there is no need to assign a mixed type to it.
So the main question is the cause of that fault message. Looking at the code there is only one occurance where the state is written with the dafault value (from io-package.json or from the admin page.
Admin page has also number
"fritz_windowtime": {
"type": "number",
"label": "default time (minutes) for window open activation",
"newLine": false,
"default": 5
},

So I am not sure what causes the error.

Maybe the datapoints are still from the creation from a very old version? They are not overwritten when a new version assigns different type, min, max etc., because already existing.
Therfore the concerned datapoints have to be deleted when the adapter is stopped. With a new start of the adapter the datapoints are created with their "new" settings.

Maybe you can point out where the string storing happens.

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.

2 participants