Skip to content

Basic Nimbus support#103

Merged
fustom merged 7 commits intofustom:mainfrom
domenicoblanco:main
Sep 13, 2023
Merged

Basic Nimbus support#103
fustom merged 7 commits intofustom:mainfrom
domenicoblanco:main

Conversation

@domenicoblanco
Copy link
Copy Markdown
Contributor

Hi! I used this library for a while and finally decided to inspect the Ariston APIs to find how the cooling consumptions are calculated. Unluckily the energyAccount endpoint returns an empty list, so I tried to obtain it differently.

The difference between Galevo and Nimbus is made with a value that I hope is false for Galevo devices (HPMP_SYS) as I read that are condensing boilers.

In any case, feel free to reject this PR as I understand could cause problems to other users

@fustom
Copy link
Copy Markdown
Owner

fustom commented Sep 1, 2023

Wow! Thank you! I will review your request soon.

@domenicoblanco
Copy link
Copy Markdown
Contributor Author

Thanks!

@fustom
Copy link
Copy Markdown
Owner

fustom commented Sep 12, 2023

I merged Nimus device into Galevo device. Please test the new code and let me know if everything works as expected.

@domenicoblanco
Copy link
Copy Markdown
Contributor Author

As energy_account would contain an empty list in LastMonth, it wouldn't be a None value.
With this small fix, now it works as expected :)

@fustom fustom merged commit 8db88a4 into fustom:main Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants