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

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

Closed
2 of 3 tasks
Scoop2389 opened this issue Jul 23, 2024 · 1 comment
Closed
2 of 3 tasks
Labels
bug Something isn't working

Comments

@Scoop2389
Copy link

Checklist

  • I am not using Home Assistant. Or: a developer has told me to come here.
  • I have checked the troubleshooting section and my problem is not described there.
  • I have read the changelog and my problem is not mentioned there.

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.

@Scoop2389 Scoop2389 added the bug Something isn't working label Jul 23, 2024
@Scoop2389 Scoop2389 changed the title Log nodes only supports nodes 1-232 even though Long Range Nodes are from 256-4256 Log nodes only supports nodes 1-232 even though Long Range Nodes start at 256 Jul 23, 2024
@robertsLando
Copy link
Member

@AlCalzone Should I change the check so?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants