zwave-js / zwave-js-ui

Full featured Z-Wave Control Panel UI and MQTT gateway. Built using Nodejs, and Vue/Vuetify
https://zwave-js.github.io/zwave-js-ui
MIT License
988 stars 206 forks source link

Log nodes only supports nodes 1-232 even though Long Range Nodes start at 256 #3827

Closed Scoop2389 closed 4 months ago

Scoop2389 commented 4 months ago

Checklist

Deploy method

Home Assistant Add-on

Z-Wave JS UI version

9.16.2

ZwaveJS version

13.0.2

Describe the bug

Settings -> Z-wave -> Log nodes only supports nodes 1-232 even though higher node numbers are possible due to Z-Wave Long Range. This means you cannot restrict logging to long range nodes.

To Reproduce

  1. Have a Z-wave stick/dongle/whatever that allows for Long Range Devices and connect at least one
  2. Go to Settings
  3. Z-wave Dropdown
  4. "Log nodes" Setting
  5. Type in the ID of any Z-wave Long Range Node
  6. Gives error "Nodes must be between 1-232", attempting to save gives error "Your configuration contains errors, fix it"

Expected behavior

  1. Have a Z-wave stick/dongle/whatever that allows for Long Range Devices and connect at least one
  2. Go to Settings
  3. Z-wave Dropdown
  4. "Log nodes" Setting
  5. Type in the ID of any Z-wave Long Range Node
  6. No error, saving works

Additional context

I am using Home Assistant bum bum bum. However, this is most probably likely not a Home Assistant issue. If this is a Home Assistant issue, feel free to let me know and I will happily move it to another repository.

robertsLando commented 4 months ago

@AlCalzone Should I change the check so?