search

Domochip / WirelessPalaControl

D1Mini to replace Palazzetti, Jotul, TurboFonte, Godin stove ConnectionBox
41 stars 9 forks source link

Fix non numeric value for STOVE_DATETIME #10

Closed Flobul closed 2 years ago

Flobul commented 2 years ago

To reproduce it: MQTT: activated On status web page, open web inspector, in console, when datas are refreshed, this error occurs :

Capture d’écran 2022-02-26 à 16 01 03

Domochip commented 2 years ago

Does : _statusEventSource.send((String("{\"STOVE_DATETIME\":") + (char *)STOVE_DATETIME + '}').c_str()); Fix the issue too?

Flobul commented 2 years ago

Doesn't work: Capture d’écran 2022-02-26 à 16 57 05

Domochip commented 2 years ago

arf, I think it is just missing some double-quote around like this : _statusEventSource.send((String("{\"STOVE_DATETIME\":\"") + STOVE_DATETIME + "\"}").c_str()); Isn't it?

Flobul commented 2 years ago

Solved it! Simple^^ Capture d’écran 2022-02-26 à 17 12 35

Domochip commented 2 years ago

Fixed in last commit. Thank you!