dewenni / ESP_Buderus_KM271

Control your Buderus Logamatic 2107 or HS-2105 with MQTT or Home Assistant
MIT License
64 stars 11 forks source link

ArduinoJson warning #71

Closed Franck78 closed 7 months ago

Franck78 commented 9 months ago

Not blocking the build of the final binary, but not working ;)

As you know , I'm (still) with Arduino IDE 1.8. The library manager says "ArduinoJSON 7.0.3" is available.

When I select it, repeated warning is issued:

/Arduino/km271_3.31/config.cpp:152:28: warning: 'DynamicJsonDocument' is deprecated: use JsonDocument instead [-Wdeprecated-declarations]

Result is webserver unable to get any data and displaying only "Connected".

Need to use arduino_json 6.12.5

Franck

dewenni commented 9 months ago

Thanks for the hint. I looked at this and unfortunately realized that I can't switch to ArduinoJSON v7 yet, because the ESPUI library for the WebUI still uses the functions that are no longer available from V7. So I have to wait until this library ESPUI has also been updated. We have to stay on the V6 for that long.

Franck78 commented 9 months ago

this is strange : "WebUI still uses the functions that are no longer available from V7"

When I replace as suggested, I don't have any undefined symbols!

Using WebUI ?(don't know who/what provides it). And ESPUI 2.2.3 (which have no json feature request or bug listed)

dewenni commented 9 months ago

my web interface is based on the library ESPUI and this still use DynamicJsonDocument what is no longer available in V7 https://github.com/search?q=repo%3As00500%2FESPUI%20dynamicjson&type=code

dewenni commented 8 months ago

@Franck78 Since I no longer have a dependency in the current release candidate #80, I was able to switch to ArduinoJSON V7. I am happy for any feedback of the new version

dewenni commented 7 months ago

fixed since release 4.0.0

Franck78 commented 7 months ago

Hello, trying to compile the thing.

stuck on

#include <gzip_c_css.h>
#include <gzip_js.h>
#include <gzip_login_html.h>
#include <gzip_m_css.h>
#include <gzip_main_html.h>

even google has nothing to say about those files ! Which lib is it ??

dewenni commented 7 months ago

Hi @Franck78 These are header files that are generated in PlatformIO with a python script. I use a script to generates these files based on HTML, CSS and JS source files to a compressed gzip format. If you are using Arduino IDE this could be a problem...

I have set these files to the .gitignore because the compressed data change on every build. But I can change this and add the files to the repository.

As a quick help, files attached Archive.zip

Franck78 commented 7 months ago

built and sent to my test ESPS3, ok.

Just add some comment in the header file on how to build the gzipped files.

No time to really read changes in the code. I hope you separeted mttq/interface translated strings;) I see no mini light show at startup / doubleReset not easily deactivated (with #define #undef help) I also convert burner time in something readable (only for the display, not mqtt messages). burner-time

next thing to do (for ESP-S3 plenty of room), have some graphics of interesting data, served directly, some day.

dewenni commented 7 months ago

I hope you separeted mttq/interface translated strings;) => yes I see no mini light show at startup / doubleReset not easily deactivated => no, sorry I also convert burner time in something readable (only for the display, not mqtt messages). => not a bad idea