search

lumapu / ahoy

Various tools, examples, and documentation for communicating with Hoymiles microinverters
https://ahoydtu.de
Other
947 stars 222 forks source link

[Bug] 0.8.140 Config export leads to misleading message "reboot AhoyDTU to apply all your configuration changes" #1730

Open fsck-block opened 1 month ago

fsck-block commented 1 month ago

Before exporting: AHOY-DTU2 - Index

After export of configuration AHOY-DTU2 - Index

Seems to be a side effect of

void onDwnldSetup(AsyncWebServerRequest *request) {
    AsyncWebServerResponse *response;

    // save settings to have latest firmware changes in export
    mApp->saveSettings(false);`
Gubi2023 commented 1 month ago

siehe #1721: Feature, no bug!

fsck-block commented 1 month ago

Feature, no bug!

Könntest du mir erklären wo das das Feature ist?

Ich speichere meine Änderungen und mache einen Reset. Nach dem Neustart exportiere ich nun die funktionierenen Einstellungen Ich werde aufgefordert einen Reset durchzuführen um meine Änderungen zu aktivieren. Nach dem Neustart exportiere ich nun die funktionierenen Einstellungen Ich werde aufgefordert einen Reset durchzuführen um meine Änderungen zu aktivieren. ...

Gubi2023 commented 1 month ago

wurde hier #1454 schon vor 6 Monaten ausführlich diskutiert!

fsck-block commented 1 month ago

ja ihr hab es gefunden 😊. Werde ich erst mal nicht ändern, wollte an den Einstellungen sowie einiges umbauen, sodasss nur noch selten ein Neustart erforderlich ist, bis dahin bleibt es so wie es ist.

Wenn ich @lumapu richtig verstanden habe, ist das kein Feature und er wollte #1454 offen lassen bis die Einstellungen umgebaut hat. Na dann lassen wir doch diesen Issue solange offen :wink: