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

[insteon] Fix imeter solo product first record location #18075

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

Conversation

jsetton
Copy link
Contributor

@jsetton jsetton commented Jan 9, 2025

This change corrects the first database record location configuration for iMeter Solo products.

This fix should be backported as the database verification for iMeter Solo products is currently failing leaving the thing in pending configuration status.

@jsetton jsetton added the bug An unexpected problem or unintended behavior of an add-on label Jan 9, 2025
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/4-3-1-insteon-binding-does-not-properly-read-the-plm-all-link-database/161266/11

@jlaur jlaur added the additional testing preferred The change works for the pull request author. A test from someone else is preferred though. label Jan 9, 2025
@jlaur
Copy link
Contributor

jlaur commented Jan 10, 2025

@jsetton - I added the "additional testing preferred" after checking your recent posts in the referenced thread. However, I can't keep up with the thread, so please remove the label once the testing has been confirmed and the PR is ready to be merged.

@jsetton
Copy link
Contributor Author

jsetton commented Jan 10, 2025

The testing has been confirmed.

@jsetton jsetton removed the additional testing preferred The change works for the pull request author. A test from someone else is preferred though. label Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants