forked from ginkage/MHI-AC-Ctrl-ESPHome
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Using the CLIMATE_FAN_QUIET instead of DIFFUSE #1
Comments
I think that would be a breaking change at Esphome if you were to insert a new value bumping the other +1. |
Wouldn't it be better to use the CLIMATE_FAN_QUIET which is already there, and update HomeAssistant add quiet? |
oh, i think it now is in home assistant: |
Bill-Dung
pushed a commit
that referenced
this issue
Jun 29, 2023
rename diffuse to quiet as quiet is now supported by HomeAssitant
Everything's done for me |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ervee commented
Perhaps using the CLIMATE_FAN_QUIET instead of DIFFUSE. CLIMATE_FAN_QUIET should also be available in climate and esphome and is perhaps a more logical choice for ultra-low fan speed.
CLIMATE_FAN_QUIET is not be showing up in Home Assistant Card like CLIMATE_FAN_MIDDLE and CLIMATE_FAN_FOCUS.
I already opened a feature request at ESPHome: esphome/feature-requests#2149
The text was updated successfully, but these errors were encountered: