Closed thecadams closed 7 months ago
Tested by adding my own repo to HACS and it's working (unrelated there is the power socket showing the heater running at 1800W..)
And can I just say thank you for the quality of the code as well! It was very easy to test with just the API and very easy to see how to add my changes and test, as you can see I went from knowing nothing as a first-time HA users, to raising a ticket, to making a PR, to demonstrating it working, in only a couple hours.
Very cool. Thanks for the great work on the module!
Hi @thecadams, thanks for your contribution and kind words 😄
I've made some minor tweaks for code style, but one thing to note is that I've removed the proposed errors sensor. I've seen the same error values for a number of other models, but since errors rarely happen, nobody has ever been able to confirm exactly how the API values correspond to what's seen on the spa display. I'm trying to avoid adding code based on guesswork, so we'll leave that out for now.
I'll force this through ignoring the unrelated test failure.
@cdpuk you’re welcome - and that's a shame about removing the error handling, because I’ve had the E02 happen against my fork with the PR as submitted (flow rate sensor tripped due to a dirty filter) and it was caught and my automation triggered, which (unrelated to this PR) switched off power to the tub.
So I do think the code is correct as submitted and strongly recommend adding that error handling back in.
Either way thanks again for your great work on this module!
With this script:
I got the following output for my Hydrojet Pro:
Almost all the fields referenced by the Hydrojet code are there - but the error codes are different, so I added a new error code sensor. I only included the codes in the manual at https://support.bestwayaftersales.co.uk/wp-content/uploads/2020/01/Maldives-HJ-Pro-2020.pdf , although my API output didn't have
earth
so I've made it not raise if that key is missing.I'd like to test this in my HA setup, do I just add my own repo as a custom repo? Or are there extra steps like publishing it to HACS?