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.

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

Allow HOT_WATER device types #126

Merged
merged 1 commit into from
Oct 3, 2021
Merged

Allow HOT_WATER device types #126

merged 1 commit into from
Oct 3, 2021

Conversation

eko
Copy link
Owner

@eko eko commented Oct 3, 2021

Two things in this pull request:

  • Added a new type to specify device type (HEATING, HOT_WATER) in the metrics labels
  • Support HOT_WATER device type response APIs

Here is an example output for HOT_WATER:

[2021-10-03T10:07:40Z INFO  tado_exporter] starting tado° exporter on address: 0.0.0.0:9898
[2021-10-03T10:07:40Z INFO  tado_exporter] waiting for the first tick in 10 seconds...
[2021-10-03T10:07:50Z INFO  tado_exporter::tado::client] retrieving zone details for Bureau...
[2021-10-03T10:07:50Z INFO  tado_exporter::tado::client] retrieving zone details for Cuisine...
[2021-10-03T10:07:50Z INFO  tado_exporter::tado::client] retrieving zone details for Salon...
[2021-10-03T10:07:50Z INFO  tado_exporter::tado::client] retrieving zone details for Chambre...
[2021-10-03T10:07:50Z INFO  tado_exporter::tado::metrics] -> Bureau (HOT_WATER) -> setting temperature (celsius): 55
[2021-10-03T10:07:50Z INFO  tado_exporter::tado::metrics] -> Bureau (HOT_WATER) -> setting temperature (fahrenheit): 131
...

@cristicalin
Copy link

Just tested this on an account with a hot water controller and it solves the issue.

@eko
Copy link
Owner Author

eko commented Oct 3, 2021

Great, thank you @cristicalin!

@eko eko merged commit c63e59f into master Oct 3, 2021
@eko eko deleted the allow-hotwater-devices branch October 3, 2021 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants