search

albetaCOM / esp-haier

ESP8266 code to connect to Haier Air Conditioner ((firmware version R_1.0.00/e_2.5.14) with wifi module support
53 stars 8 forks source link

Pull Futures from Oksion/esphaier fork #2

Closed AdamN1 closed 4 years ago

AdamN1 commented 4 years ago

Hi, Couple of futures like fan control is merged in the fork https://github.com/Oksion/esphaier, like:

albetaCOM commented 4 years ago

Thanks Adam!

I'll take a loop and import the changes!

AdamN1 commented 4 years ago

After adding new function README should be changed.

albetaCOM commented 4 years ago

Adam

albetaCOM commented 4 years ago

Hi Adam, I imported locally the changes but I get many errors related to the esphome climate traits: src/Haierv2.h:168:16: error: 'class esphome::climate::ClimateTraits' has no member named 'set_supports_dry_mode' traits.set_supports_dry_mode(true); ^ src/Haierv2.h:169:16: error: 'class esphome::climate::ClimateTraits' has no member named 'set_supports_fan_only_mode' traits.set_supports_fan_only_mode(true); ^ src/Haierv2.h:170:16: error: 'class esphome::climate::ClimateTraits' has no member named 'set_supports_fan_mode_on' traits.set_supports_fan_mode_on(false); ^ src/Haierv2.h:171:16: error: 'class esphome::climate::ClimateTraits' has no member named 'set_supports_fan_mode_off' traits.set_supports_fan_mode_off(false); ^ src/Haierv2.h:172:16: error: 'class esphome::climate::ClimateTraits' has no member named 'set_supports_fan_mode_auto' traits.set_supports_fan_mode_auto(true);

Do you have a different version of ESPHome that already support those traits?

albetaCOM commented 4 years ago

I've changed to ESPHome developer version v1.15.0b4 and it is working now. Thanks!

In the next days I'll try to adapt every new feature so it works with everything available.

AdamN1 commented 4 years ago

Sorry fo delay, yes you need beta (15) version of esphome.