[ICP-13654] Added specific UI metadata for Qubino dimmers #52444
Conversation
| */ | ||
| metadata { | ||
| definition(name: "Qubino Dimmer", namespace: "qubino", author: "SmartThings", mnmn: "SmartThings", vid:"generic-dimmer-power-energy", ocfDeviceType: "oic.d.switch", runLocally: false, executeCommandsLocally: false) { | ||
| definition(name: "Qubino Dimmer", namespace: "qubino", author: "SmartThings", mnmn: "SmartThings", vid:"qubino-dimmer-power-energy", ocfDeviceType: "oic.d.switch", runLocally: false, executeCommandsLocally: false) { |
There was a problem hiding this comment.
What's different about this metadata?
There was a problem hiding this comment.
I thought @KKlimczukS 's changes made it so that despite the device's internal range being 2-100, we scaled our commands so that the user still saw 1-100?
There was a problem hiding this comment.
Yes, Konrad set minimum dimming level as 2% in the device configuration, but because device starts recalculating those values when the range is different we had to remove this configuration (ICP-13654, ICP-13361). We decided to set this range by UI metadata.
There was a problem hiding this comment.
@greens We had to find another approach/workaround and constraint those min dimming values from the application (DTH and UI) side instead of setting this configuration parameter on the device - that's was the only way to make it work and avoid dimmer level jumping (on the UI slider, in a quite non deterministic way).
There was a problem hiding this comment.
okay, well then whenever you're ready @MWierzbinskaS
…sCommunity#52444) * ICP-13654 Added specific UI metadata for qubino dimmers * Added custom metadata for qubino flush dimmer 0-10V
@SmartThingsCommunity/srpol-pe-team
@greens
Could you please take a look at the code?