zwave-js / node-red-contrib-zwave-js

The most powerful, high performing and highly polished Z-Wave node for Node-RED based on Z-Wave JS. If you want a fully featured Z-Wave framework in your Node-RED instance, you have found it.
MIT License
47 stars 6 forks source link

Text box not showing for all config options #74

Closed crxporter closed 3 years ago

crxporter commented 3 years ago

Plugin Version 3.8.0.

Platform Raspberry pi

Describe the bug When entering some config params, there is only an option for predefined options when there should also be an option for any value. For example the Aeotec 3 in 1 sensor.

User manual shows this for param 1: image

But node red UI shows only the option to disable: image

I would propose another option in there for "Other" like this: image

I believe it should be fine on any config param since the device will simply ignore something out of bounds, right?

marcus-j-davies commented 3 years ago

Yup.

Its a known bug in the UI. A text box should appear. IF

allowManualEntry is set to true on a param (and it does) however, if there are also predefined options as well as allowManualEntry being true, it wont show the text box.

I will take a look at this over the weekend (it should be an easy fix) - hopefully for V4