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.

Sign up for GitHub

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

Jump to bottom

MQTT doesn't report temperature sensors unless at least 1 IO is enabled #66

Closed
nielsonm236 opened this issue May 1, 2021 · 2 comments
Closed

MQTT doesn't report temperature sensors unless at least 1 IO is enabled #66

nielsonm236 opened this issue May 1, 2021 · 2 comments

Comments

@nielsonm236
Copy link
Owner

Reported by jmcvieira1: Disable all i/o port´s leave mqtt and DS18B20 turned on save and reboot, and the module will not connect to the mqtt server. I have to enable one port to the module connect to the server again.
@nielsonm236
Copy link
Owner Author

I found a check in code where I was requiring at least one I/O pin be enabled for MQTT to connect. I'm not sure why that check was originally put in the code other than the boot process is quicker if there are no resources enabled ... but I've removed the check and now MQTT will connect even if no I/O pins are enabled. Now if the only thing enabled is the DS18B20 sensors MQTT will connect properly and the temperature sensors display properly in HA.

FYI, I rebooted with all I/O disabled and the DS18B20 checkbox "off". The MQTT connect process completes as it should even though there are no resources enabled on the Network Module.

This fix will be applied in the next release.

nielsonm236 added a commit that referenced this issue May 9, 2021
@nielsonm236
Numerous fixes tested for release
2259e91 
Fixed Issue #55: Home Assistant; Auto Discovery; MQTT; Temp Sensors do not get deleted and redefined properly when added/removed
Fixed Issue #56: Temperature Sensor accuracy improvement
Fixed Issue #57: Temperature sensor order in GUI and Home Assistant. Fixed by changing Temperature Sensor ID’s from “1, 2, 3, 4, 5” to a 12 character ID per sensor that is based on the sensor serial number
Fixed Issue #58: Home Assistant; Auto Discovery; MQTT; Should a temperature sensor be deleted in HA if it fails during runtime?
Fixed Issue #62: Browser Only; Timers don’t work with Invert setting. Included in fix is a correction for “Timer values not saved correctly” and “Timer value being set to 256 in error”.
Fixed Issue #63: REST commands preventing IO Control page from making pin state changes.
Fixed Issue #66: MQTT doesn’t report temperature sensors
@nielsonm236 nielsonm236 mentioned this issue May 9, 2021
Merged
@nielsonm236
Copy link
Owner Author

Fixed with release 20210509 2031

@nielsonm236 nielsonm236 mentioned this issue Sep 14, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nielsonm236