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

Send Debug Messages to MQTT #40

Merged

Conversation

Snuffy2
Copy link
Collaborator

@Snuffy2 Snuffy2 commented Nov 23, 2023

Fixes #37

Creates a Last Debug Message sensor:

  • Goes to the Diagnostics section for the JuiceBox device in HA
  • Is disabled by default
  • Message is active for 60 seconds before sensor resets to Unavailable
  • Sends a Startup message when JPP Starts
2023-11-23_22-05-10 957 2023-11-23_22-29-27 148 2023-11-23_22-24-06 995

@snicker
Copy link
Collaborator

snicker commented Nov 23, 2023

we should publish this to a different sensor rather than the main status sensor. Status should only reflect the state of the charger (Plugged In, Charging, Unplugged, Error)

Debug/Info messages could go to a "Last Debug Message" sensor or something to that effect? perhaps that entity is disabled by default in Home Assistant (though not sure if MQTT auto discovery supports this)

@Snuffy2
Copy link
Collaborator Author

Snuffy2 commented Nov 23, 2023

Makes sense. I'll see what I can do.

v0.2.0 doesn't need to wait for this in my opinion.

@Snuffy2 Snuffy2 force-pushed the Send-Debug-Messages-to-MQTT-Status branch from 726675d to 8b3eb7e Compare November 24, 2023 03:14
@Snuffy2 Snuffy2 changed the title Send Debug Messages to MQTT Status Send Debug Messages to MQTT Nov 24, 2023
@Snuffy2 Snuffy2 force-pushed the Send-Debug-Messages-to-MQTT-Status branch from 8b3eb7e to 341ec09 Compare November 24, 2023 03:28
@Snuffy2 Snuffy2 requested a review from snicker November 24, 2023 17:32
@snicker snicker merged commit f96f52d into JuiceRescue:master Nov 25, 2023
@Snuffy2 Snuffy2 deleted the Send-Debug-Messages-to-MQTT-Status branch December 11, 2023 12:26
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.

UnboundLocalError when receiving DBG,ERR:JCom_TxTcpData_Wlan
2 participants