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

[insteon2] Initial contribution #16761

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

jsetton
Copy link
Contributor

@jsetton jsetton commented May 13, 2024

This is a rewrite of the Insteon binding adding some much needed improvements while keeping some of its core code refactored. Apart from simplifying the user experience by retrieving all the configuration directly from the device when possible, and improving the way the Insteon things are configured in MainUI, here is an exhaustive list of the changes:

  • introduced device configuration automated determination
  • converted mode-based number items to string type with descriptive states
  • added number items uom support
  • added distinct scenes and x10 device things
  • added distinct bridge things for supported hub/plm devices
  • added button event trigger channels
  • added device products configuration layer
  • added device link database support
  • added device cache storage
  • added device operating flags controls
  • added modem configuration flags controls
  • added related devices synchronization feature
  • added heartbeat timeout monitor
  • added ability to link/unlink a device to the modem
  • added ability to add missing default links
  • added link database & scene management support
  • added scene state support
  • added new i3 devices basic support
  • added ezrain sprinkler device support
  • revamped console commands with auto-completion support
  • improved discovery service
  • improved thing status

Since the configuration process was revamp, this is a breaking change with regards to existing user of the current binding requiring them to reconfigure their Insteon setup. This is why I am submitting a new separate binding with the expectation that the current one would be renamed as legacy allowing existing users to keep using that binding until they decide to move to the new one.

@jsetton jsetton force-pushed the insteon2 branch 2 times, most recently from 4cee278 to dd98c56 Compare May 16, 2024 00:27
@lolodomo lolodomo added the new binding If someone has started to work on a binding. For a new binding PR. label May 19, 2024
Signed-off-by: jsetton <jeremy.setton@gmail.com>
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants