Bring back the boiler status

[cgtobi]

Description:

In the past the state returned whether the the device was heating or idle. Now the state returns the operation mode which can be auto, manual, etc. and is a different thing.

The boiler_status attribute returned the operation mode which has changed to the actual mode not the state of the boiler.
This PR brings back the actual boiler status.

Related issue (if applicable): fixes #

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>

Example entry for configuration.yaml (if applicable):

Checklist:

• The code change is tested and works locally.
• Local tests pass with tox. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

• New dependencies have been added to the REQUIREMENTS variable (example).
• New dependencies are only imported inside functions that use them (example).
• New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
• New files were added to .coveragerc.

If the code does not interact with devices:

• Tests have been added to verify that the new code works.

[awarecan]

Off-topic, why we still have a boiler hide in climate component?

[shanbs]

Important bug fix, should be merged.

[shanbs]

Off-topic, why we still have a boiler hide in climate component?

Many heating systems do have a boiler.

[awarecan]

We have water_heater component designed for this purpose. And I thought we already moved all of those devices to water_heater

[shanbs]

As the name indicates, I think it is for water heater, not the house heating system. A house heating does not have minimal temperature, maximum temperature setting, usually. Instead, it should have target temperature, current temperature, etc. So at least for the moment, climate is more suitable for such a heating system.

[cgtobi]

This is no water heater and a fundamental part of a heating system, so in my opinion it is perfectly valid to keep this as a property.

See the Netatmo API documentation, they expose for a reason.
https://dev.netatmo.com/resources/technical/reference/energy/homestatus

[awarecan]

OK, I see.

[colsonpete]

A reason for needing boiler status (I think!)...
I have a Netatmo Thermostat. I have a stand-alone CH pump which is necessary for a new underfloor heating circult in a recently added extension. But how do I control it? For the last couple of years I have used a separate wireless thermostat connected to a boiler control which simply turns the pump on and off depending on the room temperature. It would be far better to be able to interrogate the Netatmo Thermostat and if it is calling for heat, turn the pump on, if not calling for heat, turn pump off. This is the prime reason I recently started my Hass.io project.

Having access to a property that tells me whether or not the netatmo thermostat is calling for heat (e.g. blue light on at base unit) is exactly what I need! Please bring it back!

[MartinHjelmare]

It is back since this PR is merged.

For more discussion see below.

If you want to suggest an enhancement please open a feature request in the Feature Requests section of our community forum.

Merged PRs should not be used for enhancement discussion or bug reports. If you've found a bug it's ok to make a review with inline comments and link to an issue that reports the bug.

Thanks!