switch for auxheat missing about mbapi2020 HOT 5 CLOSED

hi @danielrheinbay ,

I have added the services into the master branch. Please check the services.

Could you share the attributes and values that you get, please. I need this for the docs.

from mbapi2020.

Thanks for testing, @danielrheinbay

I have removed the outlet device_class from the switch again.

I will take a look on the climate component, but with lower Prio.

from mbapi2020.

Thanks Daniel, I’ll merge your PR later today. Currently, only the preheat and not the aux_heat services are implemented. Are you willing to test if I implement the auxheat services on on a special branch? If this works then the switch is simple.

from mbapi2020.

Thank you for clarifying, @ReneNulschDE. Absolutely willing to test :)

from mbapi2020.

...and you've even implemented the switch in fb54579 -- thank you so much!
I have tested the switch and it works for me. It can turn the aux heat on and off.

sensor.<car name>_auxheat_status has these attributes:
car:
vin:
value_short: inactive
value_description: inactive
retrievalstatus: VALID
timestamp: '2021-01-31T10:08:48'
auxheattime1: '14:15' (i.e. I've set this some days ago)
auxheattimeselection: '0' (i.e. the time selection isn't actually active, as it was used a few days ago)
auxheattime3: '00:00'
auxheatActive: false
auxheatwarnings: '0'
auxheattime2: '00:00'
friendly_name: Auxheat Status
icon: 'mdi:radiator'

switch.<car name>_auxheat has these attributes:
car:
vin:
retrievalstatus: VALID
timestamp: '2021-01-31T10:08:55'
friendly_name: AuxHeat
device_class: outlet

The last attribute, device_class, got me thinking: For now, switching the aux heat on and off works, which is awesome. However, Home Assistant sees the aux heat as an electrical outlet. Unfortunately, switches don't support the device_class attribute. So may be at some point we can implement a Climate component instead, see https://www.home-assistant.io/integrations/climate/ and https://developers.home-assistant.io/docs/core/entity/climate/ for details. Maybe this would allow for abstraction of auxheat and preconditioning...? What are your thoughts, @ReneNulschDE?

from mbapi2020.