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

[iaqualink] Device with SerialID not connecting #12654

Closed
mdsolutech opened this issue Apr 25, 2022 · 3 comments · Fixed by #12659
Closed

[iaqualink] Device with SerialID not connecting #12654

mdsolutech opened this issue Apr 25, 2022 · 3 comments · Fixed by #12659
Labels
bug An unexpected problem or unintended behavior of an add-on

Comments

@mdsolutech
Copy link

Hello guys!

Expected Behavior

The device should be "ONLINE".

Current Behavior

The device is "OFFLINE" with error: "CONFIGURATION_ERROR: No Device for given serialId found".
"log:set DEBUG org.openhab.binding.iaqualink" does not provide any information (no logs).

Possible Solution

None

Steps to Reproduce (for Bugs)

  1. Connect a controller without SerialID => OK
  2. Connect a controller with SerialID => KO

Context

I am trying to connect to 3 controllers through one account by providing a serial ID in the config or dynamic thing creation.
I am not able to control 2 of my controllers (1 is available by default without specifying serial ID).

Your Environment

OpenHAB 3.2.0, running on Mac, installed using official installer.

  • Version used: OpenHAB 3.2.0, iAqualink 3.2.0
  • Environment name and version (e.g. Chrome 76, Java 8, Node.js 12.9, ...): Chrome 100.0.4896.88, OpenJDK 11.0.13
  • Operating System and version (desktop or mobile, Windows 10, Raspbian Buster, ...): Mac Big Sur 11.6.5

Thanks for the help!

@mdsolutech mdsolutech added the bug An unexpected problem or unintended behavior of an add-on label Apr 25, 2022
digitaldan added a commit to digitaldan/openhab-addons that referenced this issue Apr 25, 2022
Fixes openhab#12654

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
@digitaldan
Copy link
Contributor

Hi, so yeah i put in the code to support multiple pools a while ago in anticipation of it being needed, but don't use it personally . I'm not sure if it ever worked, or likely the api changed slightly which broke how we search for serial ids ( i assumed i tested it).

I have an updated jar at https://github.com/digitaldan/openhab-addons/releases/download/iaqualink-serialid-fix/org.openhab.binding.iaqualink-3.3.0-SNAPSHOT.jar if you want to give it a try. You can either drop this in your addons folder (remove the old binding first) or download it somewhere on your openhab box and udpate the binding in place using the openHAB CLI with the command:

bundle:update org.openhab.binding.iaqualink file:////tmp/userdata/org.openhab.binding.iaqualink-3.3.0-SNAPSHOT.jar
assuming you put it in the /tmp folder

@mdsolutech
Copy link
Author

Damn you're good!

That works perfectly!

I normally use the addons.cfg file for installing the bindings I need.
How does the release of this 3.3.0 works? When will it be installed/updated directly from the addons.cfg file?

Thank you very much for your quick reply and fix!

@digitaldan
Copy link
Contributor

I'll submit a PR probably tonight or tomorrow, and it will be available in the nightly builds shortly, as well as any future milestone releases and official releases.

digitaldan added a commit to digitaldan/openhab-addons that referenced this issue Apr 27, 2022
Fixes openhab#12654

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
lolodomo pushed a commit that referenced this issue Apr 27, 2022
Fixes #12654

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this issue Apr 27, 2022
Fixes openhab#12654

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
andan67 pushed a commit to andan67/openhab-addons that referenced this issue Nov 6, 2022
Fixes openhab#12654

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
andrasU pushed a commit to andrasU/openhab-addons that referenced this issue Nov 12, 2022
Fixes openhab#12654

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
psmedley pushed a commit to psmedley/openhab-addons that referenced this issue Feb 23, 2023
Fixes openhab#12654

Signed-off-by: Dan Cunningham <dan@digitaldan.com>
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 a pull request may close this issue.

2 participants