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

Upgraded to homie 3.0 convention and one connection per device for homie #119

Merged
merged 2 commits into from
Mar 15, 2021
Merged

Conversation

chrostek
Copy link
Contributor

Solves #57

@NorbertHD
Copy link

@chrostek thank you for your contribution, but I get an exception log entry in openhab with your code and the sensor item shows offline

2020-05-20 17:25:24.370 [hingStatusInfoChangedEvent] - 'mqtt:homie300:mosquitto:crassulaovata' changed from INITIALIZING to OFFLINE (COMMUNICATION_ERROR): java.lang.Exception: Did not receive mandatory topic value: homie/crassulaovata/$name

@chrostek
Copy link
Contributor Author

chrostek commented May 20, 2020

I think it is a bug in openHAB - i have this error with other homie implementations too (volkszaehler/mbmd).
openHAB needs to receive this topic after the device was added or after openhab started - please restart miflora-mqtt-daemon after adding the device and the error will disappear

Edit: Found this:
openhab/openhab-addons#7252
so it is related to openHAB, not to my code here

@ThomDietrich ThomDietrich merged commit 7a9a7aa into ThomDietrich:master Mar 15, 2021
@ThomDietrich
Copy link
Owner

@chrostek Thanks! Really sorry this one got lost in the past. I've merged it now but I have no environment to test atm. Please let me know when you get around testing it!

@chrostek
Copy link
Contributor Author

@ThomDietrich thanks! i use/test this since May 2020 - it works without any issues with the latest openhab versions

@ThomDietrich
Copy link
Owner

Great!
As you might know from #111 I was planning to move to a different software structure. I am not sure when I'll have the time to do so but I'd be happy if I could refer back to you for improvements and testing?

Best!

@chrostek
Copy link
Contributor Author

i am no developer, i did only a bit copy & paste ;-)
but i am happy to help you with testing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants