Is this library still active in updates?

[Dcmwrnd]

Hi Tolentino,

Sorry to ask is this library still active in updates?
Because recently there are updates on Arduino IDE, ESP32 Board Manager, ArduinoJson library and also the Telegram app.

It seems that the ArduinoJson library is complaining about this line below during compiling…

ArduinoJson\src/ArduinoJson/compatibility.hpp:125:58: note: declared here
125 | class ARDUINOJSON_DEPRECATED("use JsonDocument instead") DynamicJsonDocument

AsyncTelegram2\src\AsyncTelegram2.cpp:921:44: warning: 'DynamicJsonDocument' is deprecated: use JsonDocument instead [-Wdeprecated-declarations]
921 | DynamicJsonDocument root(m_JsonBufferSize);

And also the examples – ESP_OTA – WithPassword.ino does not work anymore.

The followings are used for compiling: -

Hardware:
Mcu: ESP32 DevKitC

Compile with:-
OS: Windows 11
Arduino IDE: v2.3.8
ESP32 Board Manager: v3.3.8
esptool (compiler): v5.2.0
AsyncTelegram2.h: v2.3.3

Regards,
David

[cotestatnt]

Hi @Dcmwrnd, the repository is still active, but unfortunately, I've had very little free time lately to dedicate to this project, which is why it has remained unupdated for a while.
Regarding the DynamicJsonDocument class, I thought I had fixed every instance using #if ARDUINOJSON_VERSION_MAJOR > 6, but clearly I missed something.

I'll check and try to fix it as soon as I can.

[Dcmwrnd]

Fully understood and thanks for your reply. This is really a wonderful library. With regards to the WithPassword.ino example, I guess the recent update of Telegram app make it stop working because it was working on previous version.

Regards,
David

[cotestatnt]

Hi @Dcmwrnd
The bugs you reported should be resolved in the latest release.

[Dcmwrnd]

Oh yes, the ‘WithPassword.ino’ is running perfectly now but the compiler still have some complain if you turn on the ‘Compiler warnings’ on the ‘Preferences’ to All, otherwise you will not be able to see the warning.

WithPassword.ino:114:37: warning: format '%d' expects argument of type 'int', but argument 5 has type 'int32_t' {aka 'long int'} [-Wformat=]
114 | snprintf(report, 128, "Start firmware update\nFile name: %s\nFile size: %d",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
115 | msg.document.file_name.c_str(), msg.document.file_size);
| ~~~~~~~~~~~~~~~~~~~~~~
| |
| int32_t {aka long int}

To eliminate the warning, I replaced with: -
snprintf(report, 128, "Start firmware update\nFile name: %s\nFile size: %ld", msg.document.file_name.c_str(), (long)msg.document.file_size);

Anyway, thank you so much for the updates even though you are on a tight schedule.

Regards,
David