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

HTTP-API - MQTT-Status not set correctly #360

Open
jwiesel opened this issue Feb 27, 2025 · 2 comments
Open

HTTP-API - MQTT-Status not set correctly #360

jwiesel opened this issue Feb 27, 2025 · 2 comments

Comments

@jwiesel
Copy link

jwiesel commented Feb 27, 2025

FW-Version: 4.04

Issue: The /check_status endpoint does not set the value for "mqtt_en" correctly.

Steps to reproduce:

  • Open the /check_status page and check the value for key "mqtt_en" - Let's assume it is disabled
  • Go to the WiCAN Settings page, enable MQTT and add the server settings. Submit the changes
  • Check, if messages appear on the MQTT Broker - (ok)
  • Open the /check_status page again and check the value for key "mqtt_en". --> The value is still disabled (not ok)
  • If I enable Battery Alert, the MQTT-Status is set to enabled as well

Potential root cause: API field "mqtt_en" might be linked to Battery Alert

@jwiesel jwiesel changed the title HTTP-API - MQTT-Status not updated HTTP-API - MQTT-Status set correctly Feb 27, 2025
@jwiesel jwiesel changed the title HTTP-API - MQTT-Status set correctly HTTP-API - MQTT-Status not set correctly Feb 27, 2025
@meatpiHQ
Copy link
Owner

meatpiHQ commented Mar 3, 2025

@jwiesel good catch! Fixed 1f0be35

@jwiesel
Copy link
Author

jwiesel commented Mar 5, 2025

Thanks a lot!

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

No branches or pull requests

2 participants