zwave_js config param should only be a toggle if there are 2 states #10812
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed change
Right now we show a toggle to the user if
enabled
/enable
anddisabled
/disable
are in the available options for the config parameter, even if there are more than two options available. We should not do that and should instead use a dropdown to show the additional states when there are more than two options available.I was not able to test this as I don't have a device that meets this criteria, but this should fix zwave-js/node-zwave-js#3836
Type of change
Example configuration
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: