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

[rainsoft] [WIP] RainSoft Binding initial contribution #14923

Draft
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

morph166955
Copy link
Contributor

This is the initial contribution of the RainSoft binding. RainSoft produces water conditioning systems for homes. A few of the newer models support statistics about water usage. This binding is mostly read-only to gather data from RainSoft.

This is still a work in progress. Binding is based off of the code of the Ring binding.

Fixes #14739

@jlaur jlaur added the new binding If someone has started to work on a binding. For a new binding PR. label May 2, 2023
Signed-off-by: Ben Rosenblum <[email protected]>
Signed-off-by: Ben Rosenblum <[email protected]>
Signed-off-by: Ben Rosenblum <[email protected]>
Signed-off-by: Ben Rosenblum <[email protected]>
Signed-off-by: Ben Rosenblum <[email protected]>
@morph166955
Copy link
Contributor Author

This is updated for 4.1.0

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As this is largely based on the ring binding, several comments i just left about the documentation (readme, xml, i18n) apply here too. Let me know when you are ready for a first pass. I'm fine if that is still in a draft state if that helps you to bring this forward.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! This is absolutely based on the ring binding. I always assumed it would go through review first, I would pull all those changes in here, and then push this for review.

@lolodomo lolodomo added the work in progress A PR that is not yet ready to be merged label May 25, 2024
Signed-off-by: Ben Rosenblum <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new binding If someone has started to work on a binding. For a new binding PR. work in progress A PR that is not yet ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[rainsoft] Add RainSoft Remind Binding
5 participants