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.

Sign up for GitHub

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

Jump to bottom

[mqtt.homeassistant] fix multi-speed fans #17813

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[mqtt.homeassistant] fix multi-speed fans #17813

wants to merge 1 commit into from
+122 −30

Commits on Nov 27, 2024

  1. [mqtt.homeassistant] fix multi-speed fans 

     * fix step math so that the state description represents the step
   scaled to 0-100%
 * introduce a format override to PercentageValue - Home Assistant
   expects integers in several places. for a true percentage, most
   devices were accepting a floating point just fine, but if there's
   a device (like a 3-speed fan) that has a command template that
   translates the speeds to some other string value, and uses a
   lookup to do that, it's important that the value we send to that
   template is an integer
 * fix that the value template for fan state is state_value_template,
   not value_template like almost everything else

Signed-off-by: Cody Cutrer <[email protected]>
    @ccutrer
    ccutrer committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    c61bce9 View commit details
    Browse the repository at this point in the history