GladysAssistant / Gladys

A privacy-first, open-source home assistant
https://gladysassistant.com
Apache License 2.0
2.57k stars 278 forks source link

MQTT external broker option is not taken in account #2040

Closed cicoub13 closed 5 months ago

cicoub13 commented 6 months ago

Describe the bug Button "Utiliser un broker dans un conteneur Docker." on MQTT Setup page is not taken in account.

When you click on the button, action updateConfigurationin index.js has an undefined variable config. So it's not taken in account and not saved into backend configuration.

image

https://github.com/GladysAssistant/Gladys/blob/master/front/src/routes/integration/all/mqtt/setup-page/index.js#L44

Your Gladys installation (please complete the following information):

Desktop (please complete the following information):

Additional context https://community.gladysassistant.com/t/gladys-et-broker-mqtt-externe/8741

McFlyPartages commented 5 months ago

Hy,

I add my config with thesame problem.

Your Gladys installation (please complete the following information):

Desktop (please complete the following information):

Pierre-Gilles commented 5 months ago

I tested extensively without managing to reproduce :/

https://community.gladysassistant.com/t/gladys-et-broker-mqtt-externe/8741/28?u=pierre-gilles

Pierre-Gilles commented 5 months ago

Fixed in https://github.com/GladysAssistant/Gladys/pull/2046