evcc icon indicating copy to clipboard operation
evcc copied to clipboard

Published 20 hours ago • verified publisher icon evcc-io

add rest api for enable and disable threshold

Open StefanSchoof opened this issue 2 years ago • 2 comments

Fixes #5516

Ich habe kein MQTT Setup und habe daher die MQTT API ignoriert.

Bei der http api gibt es immer auch ein get. Ich habe aber nicht verstanden wie die aufgerufen wird. Ich denke gerade das L124 nicht erreicht werden kann, da err entweder nil oder nicht nil sein muss. https://github.com/evcc-io/evcc/blob/80ff55169993213a47a03d6353081a6621024bd6/server/http_handler.go#L110-L126 Daher ist der get nicht getestet.

  • [x] PR für docs: https://github.com/evcc-io/docs/pull/257
  • [ ] api/enablethreshold vs api/enable/threshold?

StefanSchoof avatar Dec 20 '22 17:12 StefanSchoof

@StefanSchoof dein erwähnter Code ist korrekt. Wenn strconv.ParseFloat keinen Fehler wirft. wird err = set(val) aufgerufen. D.h. es wird jetzt versucht den Wert zusetzen und der Erfolg wird in err gespeichert. Und damit kommt auch hier dann Zeile 119 zum Zug. Wenn das Setzen erfolgreich war geht er in Zeile 124, und wenn Zeile 114 ODER Zeile 116 einen Fehler hat, geht er in 120.

DerAndereAndi avatar Dec 20 '22 19:12 DerAndereAndi

Ah danke. Jetzt verstehe ich. Es wird nach dem setzen der Wert ausgelesen und zurück gegeben. Ich dachte es gebe einen Weg nur den get aufzurufen. Somit habe ich bei meinen Tests auch die getter getestet

StefanSchoof avatar Dec 20 '22 20:12 StefanSchoof

Wir haben uns entschlossen, die Apioberfläche für diese Detailparameter nicht zu vergrößern. Unser Fokus liegt weiterhin darauf, gute Automatiken zu bauen und uns nicht in Details zu verzetteln. Danke für den PR!

andig avatar Jan 22 '23 13:01 andig