Skip to content

Commit

Permalink
fix: allow State as a valid HA option for 6 States with No Data options
Browse files Browse the repository at this point in the history
Not being a HA user and a beginner in ZWave JS, this is highly likely not correct, but I can handle that.

This is a potential hack to address #6584.  For the 6 states that allow the "No Data" option as state parameters, the device mfr might not provide a state parameter (optional per the ZW spec) and in this case ZWave JS sends the state, not the state parameter.  This throws an HA validation error.  The intent of this PR is to add the "state" to the "state parameter" list so that validation will succeed. Conveniently all the "states" are higher than any Zwave "state parameter" options, so the device will never send that "state parameter".  It is only to inform HA that the value is allowed and means no data was provided.

Signed-off-by: Bob Eckhoff <[email protected]>
  • Loading branch information
apella12 committed Jan 12, 2024
1 parent fb93b59 commit 8a0c5ec
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions packages/config/config/notifications.json
Original file line number Diff line number Diff line change
Expand Up @@ -300,7 +300,8 @@
"0x01": "No data",
"0x02": "Below low threshold",
"0x03": "Above high threshold",
"0x04": "Max"
"0x04": "Max",
"0x06": "No data"
}
}
}
Expand All @@ -317,7 +318,8 @@
"0x01": "No data",
"0x02": "Below low threshold",
"0x03": "Above high threshold",
"0x04": "Max"
"0x04": "Max",
"0x07": "No data"
}
}
}
Expand All @@ -333,7 +335,8 @@
"values": {
"0x01": "No data",
"0x02": "Below low threshold",
"0x03": "Above high threshold"
"0x03": "Above high threshold",
"0x08": "No data"
}
}
}
Expand All @@ -349,7 +352,8 @@
"values": {
"0x01": "No data",
"0x02": "Below low threshold",
"0x03": "Above high threshold"
"0x03": "Above high threshold",
"0x09": "No data"
}
}
}
Expand Down Expand Up @@ -1248,7 +1252,8 @@
"0x01": "No data",
"0x02": "Below low threshold",
"0x03": "Above high threshold",
"0x04": "Max"
"0x04": "Max",
"0x05": "No data"
}
}
}
Expand All @@ -1265,7 +1270,8 @@
"0x01": "No data",
"0x02": "Below low threshold",
"0x03": "Above high threshold",
"0x04": "Max"
"0x04": "Max",
"0x06": "No data"
}
}
}
Expand Down

0 comments on commit 8a0c5ec

Please sign in to comment.