Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature Request: JSON Settings upload? #551

Closed
DennisXK opened this issue Jan 5, 2023 · 11 comments
Closed

Feature Request: JSON Settings upload? #551

DennisXK opened this issue Jan 5, 2023 · 11 comments
Assignees
@lumapu
Labels
enhancement New feature or request fixed dev fixed

Comments

@DennisXK
Copy link

DennisXK commented Jan 5, 2023

Wie bzw. wo kann man die Settings JSON wieder hochladen? Denn es ist ziemlich nervig nach jedem update immer die ganzen Einstellungen wieder vorzunmen.
@TB-79
Copy link

TB-79 commented Jan 5, 2023

Bei meinem letzten Update sind die Einstellungen erhalten geblieben. Wenn man über Update in ahoy geht .
Oder flasht du jedesmal über das Tool neu?

@mroenne2022
Copy link

@DennisXK @TB-79
Ich hab kürzlich bei 2 DTU`n ein Update (auf 0.5.66) direkt im Menü von der Ahoy durchgeführt. Die Einstellungen zum WLAN und Inverterdaten blieben ebenfalls erhalten ! Beide waren anschliessend wieder über die IP in meinem WLAN erreichbar.

@knickohr
Copy link

knickohr commented Jan 5, 2023

Die Einstellungen bleiben eigentlich schon ab der DEV 3x erhalten wenn man OTA flasht. Gehen nur verloren wenn irgendwas beim Update in die Hose geht. Bei mir war es meist das Wurstfingersyndrom, indem ich öfters auf den Update-Button gekommen bin. Aber das ist auch schon gefixed und kann nicht mehr vorkommen.

Natürlich wäre der Import des JSON wünschenswert wenn doch mal wieder was schief geht oder man auf andere Hardware umschwenkt 😉

@stefan123t
Copy link
Collaborator

@lumapu das sollte doch eigentlich mit einem POST an /save und der alten settings.json als Content-Type: application/json funktionieren, oder ?
Der /save endpoint ist an showSave() bzw. über die app und das appInterface and saveSettings() in der settings.h gebunden ist.
Dort wird dann der JSON Payload entsprechend ins FileSystem / die Config Datei geschrieben.

Am besten könnte man das Upload Form direkt auf der Settings Seite neben dem Download Button platzieren, der den Endpoint /get_setup für den onDwnldSetup() Handler für die getSetup() Methode aufruft.

@lumapu
Copy link
Owner

lumapu commented Jan 6, 2023 •
edited
Loading

nein, aktuell akzeptiert der endpoint kein JSON, ich denke aber dass das nicht viel Arbeit ist. Man kann das JSON empfangen, evtl. paar checks durchführen und dann ins Filesystem schreiben.

@DennisXK
Copy link
Author

DennisXK commented Jan 6, 2023

@mroenne2022 bei mir klappt das updaten meistens nie, deswegen habe ich gefragt!

@lumapu
Copy link
Owner

lumapu commented Jan 6, 2023

wird eingebaut 😊

@mroenne2022
Copy link

@DennisXK : Bei etwas älteren Versionen hatte ich das auch schon das Einstellungen verloren waren. Bei meinen letzten Updates allerdings nicht, da war alles vorhanden. Lediglich Communication enable noch aktivieren, das wars. Warum es bei dir selten klappt weiß ich dann nicht.

@DennisXK
Copy link
Author

DennisXK commented Jan 6, 2023

Wollte ebene von der 0.5.41 auf die 0.5.66 updaten...da kam dann das das update fehlgeschlagen ist und er startet in 20 sec neu. Somit muste ich wieder neu flashen 🙈🤷‍♂️

@lumapu lumapu self-assigned this Jan 7, 2023
@lumapu lumapu added the enhancement New feature or request label Jan 7, 2023
lumapu added a commit that referenced this issue Jan 8, 2023
@lumapu
corrected MQTT comm_disabled #529
d8e255d 
fix Prometheus and JSON endpoints (`config_override.h`) #561
publish MQTT with fixed interval even if inverter is not available #542
added JSON settings upload. NOTE: settings JSON download changed, so only settings should be uploaded starting from version `0.5.70` #551
MQTT topic and inverter name have more allowed characters: `[A-Za-z0-9./#$%&=+_-]+`, thx: @mo Demman
improved potential issue with `checkTicker`, thx @cbscpe
MQTT option for reset values on midnight / not avail / communication stop #539
small fix in `tickIVCommunication` #534
add `YieldTotal` correction, eg. to have the option to zero at year start #512
@lumapu lumapu added the fixed dev fixed label Jan 8, 2023
@stefan123t stefan123t changed the title JSON Settings upload? Feature Request: JSON Settings upload? Jan 12, 2023
modem-man-gmx pushed a commit to modem-man-gmx/ahoy that referenced this issue Jan 15, 2023
@lumapu @modem-man-gmx
corrected MQTT comm_disabled lumapu#529
6e41a83 
fix Prometheus and JSON endpoints (`config_override.h`) lumapu#561
publish MQTT with fixed interval even if inverter is not available lumapu#542
added JSON settings upload. NOTE: settings JSON download changed, so only settings should be uploaded starting from version `0.5.70` lumapu#551
MQTT topic and inverter name have more allowed characters: `[A-Za-z0-9./#$%&=+_-]+`, thx: @mo Demman
improved potential issue with `checkTicker`, thx @cbscpe
MQTT option for reset values on midnight / not avail / communication stop lumapu#539
small fix in `tickIVCommunication` lumapu#534
add `YieldTotal` correction, eg. to have the option to zero at year start lumapu#512
@MrLee99
Copy link

MrLee99 commented Feb 7, 2023

Moin!

teste gerade mehrere ESPs...da wäre ein upload der config sehr hilfreich. Leider ist der upload button ohne Funktion (mouseover geht, keine reaktion beim drücken)
Bin ich mit dem Problem alleine?

@stefan123t
Copy link
Collaborator

@MrLee99 sollte ab 0.5.70 behoben sein.
Die aktuelle Version aus dem Development03 Branch ist 0.5.83.
Welche Version hast Du denn ?

@lumapu lumapu closed this as completed Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lumapu lumapu

Labels
enhancement New feature or request fixed dev fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@stefan123t @MrLee99 @lumapu @DennisXK @knickohr @mroenne2022 @TB-79