Repository navigation
Adds new integration [patrickk88/aquamedic-local] - #13394
patrickk88 wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XwA7uP3kHSPCxwEc9uuuET
|
Thank you for submitting your repository to HACS (Home Assistant Community Store). Your submission is in the review queue: You can view the current queue here. Pull requests are processed in the order they were created, oldest first. What to avoid during review:
If you need to make changes: About draft pull requests: What happens next:
|
frenck
left a comment
There was a problem hiding this comment.
Hi @patrickk88, thanks for the submission! 👋
One thing needs fixing before this can go in.
- The README says the DC Runner controls are untested ("untested - decoded from the public datapoint schema") and that on the EcoDrift writing is only tested with flow and frequency. The integration still writes everything else to the pump: on the DC Runner on/off, speed, feed pause and timer (
custom_components/aquamedic_local/protocol.py,_DC_WRITE, lines 507 to 512), and on the EcoDrift pump on/off, wave mode, wave form, feed pause, timer and linkage (_ECODRIFT_WRITE, lines 496 to 506). These pumps keep a tank alive, and a write that has never been seen to work on a real pump can stop the flow or change how a pump couples to the others (linkage master or slave). Payloads that are not confirmed on hardware should not be sent to a device. Please either confirm each of these writes on a real pump and say so in the README, or remove the ones you have not verified until someone has. For the DC Runner that means shipping it read only: leave out its switch, number and feed controls.
Also not blocking: once the repository is in the default list, lead the README install section with searching for it in HACS.
Once that's addressed, push a new release and re-request review.
../Frenck
Blogging my personal ramblings at frenck.dev
Or subscribe to my free bi-weekly newsletter: frenck.dev/newsletter
Checklist
Links
Link to current release: https://github.com/patrickk88/aquamedic-local/releases/tag/v0.1.0
Link to successful HACS action (without the
ignorekey): https://github.com/patrickk88/aquamedic-local/actions/runs/41996942357Link to successful hassfest action (if integration): https://github.com/patrickk88/aquamedic-local/actions/runs/41996942357
🤖 Generated with Claude Code
https://claude.ai/code/session_01XwA7uP3kHSPCxwEc9uuuET