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

fix(config): update Qubino Smart Plug 16A, parameter 41 does not exist #2009

Merged
merged 2 commits into from Mar 8, 2021
Merged

fix(config): update Qubino Smart Plug 16A, parameter 41 does not exist #2009

merged 2 commits into from Mar 8, 2021

Conversation

ualex73
Copy link
Contributor

@ualex73 ualex73 commented Mar 7, 2021

No description provided.

@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Mar 7, 2021
"41": {
"label": "Treshold time for power reporting",
"42": {
"label": "Threshold time for power reporting",
"unit": "sec",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"unit": "sec",
"unit": "seconds",

Comment on lines 128 to 129
"label": "Threshold Change in Power for reporting",
"description": "Change of power consumption [Watt] reporting threshold",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"label": "Threshold Change in Power for reporting",
"description": "Change of power consumption [Watt] reporting threshold",
"label": "Power Consumption (Watt) Reporting Threshold",

I think this should be enough.

"41": {
"label": "Treshold time for power reporting",
"42": {
"label": "Threshold time for power reporting",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"label": "Threshold time for power reporting",
"label": "Power Consumption (Watt) Reporting Interval",

@AlCalzone AlCalzone merged commit 8fa4347 into zwave-js:master Mar 8, 2021
@AlCalzone
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config ⚙ Configuration issues or updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants