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

TS110E dimmer does not expose min_brightness and max_brightness #16520

Closed
mikolajpiecuch opened this issue Feb 4, 2023 · 3 comments
Closed
Labels
problem Something isn't working

Comments

@mikolajpiecuch
Copy link

mikolajpiecuch commented Feb 4, 2023

What happened?

TS110E dimmer (_TZ3210_k1msuvg6) does not expose min_brighntess and max_brightness in frontend even state of that dimmer shows that fields:

{ "brightness": 253, "linkquality": 39, "max_brightness": 255, "min_brightness": 4, "power_on_behavior": "previous", "state": "OFF", "switch_type": "momentary" }

What did you expect to happen?

Possibility to set min_brightness for TS110E dimmer - physical switch won't be able to reduce brightness below set threshold

How to reproduce it (minimal and precise)

Add TS110E dimmer to zigbee2mqtt

Zigbee2MQTT version

1.30.0

Adapter firmware version

20211115

Adapter

CC2531

Debug log

No response

@mikolajpiecuch mikolajpiecuch added the problem Something isn't working label Feb 4, 2023
Koenkk added a commit to Koenkk/zigbee-herdsman-converters that referenced this issue Feb 4, 2023
@Koenkk
Copy link
Owner

Koenkk commented Feb 4, 2023

Added!

Changes will be available in the dev branch in a few hours from now. (https://www.zigbee2mqtt.io/advanced/more/switch-to-dev-branch.html)

@Koenkk Koenkk closed this as completed Feb 4, 2023
@jacekmarek
Copy link

jacekmarek commented Feb 6, 2023

@mikolajpiecuch I have this module _TZ3210_k1msuvg6 also but I can't use brighntess as trigger in automation. Can you check this?

@mikolajpiecuch
Copy link
Author

I found this commit removed switch type for _TZ3210_k1msuvg6 -> Koenkk/zigbee-herdsman-converters@401d1c4. I'm not sure if can add only this line
tuya.exposes.switchType()
as it can affect other devices. I guess default value for the switch type is 'momentary' and that's why it works for most users. I still use 'toggle' switch so I need to have this option set

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

No branches or pull requests

3 participants