TA2k / ioBroker.vw-connect

ioBroker Adapter for VW We connect and Skoda connect
MIT License
90 stars 22 forks source link

"Bad limit" if charging-sessions is -1 in settings #312

Closed Backes89 closed 5 months ago

Backes89 commented 5 months ago

image

In the instance setting there is the option to load the wallbox history. If the setting is -1 it should not be loaded.

If setting this to -1 the protocol gets flooded with errors like this:

image

Expected behaviour would be no error message and no charging history.

TA2k commented 5 months ago

Is this a vw id with wallbox elli?

TA2k commented 5 months ago

Can you show the log after login

Backes89 commented 5 months ago

Instance is configured as "Skoda Powerpass". Wallbox is an Elli Wallbox with Skoda Branding.

I can post the log later this evening. Currently not at home.

Backes89 commented 5 months ago

image

TA2k commented 5 months ago

The -1 means deactivating completely this mean no data is fetching the GitHub version has more logs please reinstall the GitHub version and activate debug log

Backes89 commented 5 months ago

Screenshot 2024-01-26 094224

Hab mal alles persönliche geschwärzt. Oder lieber englisch hier im Git?

TA2k commented 5 months ago

das nach dem login wäre interessant aber du kannst den wert doch auch auf 1 setzen oder

Backes89 commented 5 months ago

Screenshot 2024-01-26 102632

Natürlich kann ich da die 1 setzen. Aber in den Optionen der Instanz gibt es ja extra den Tooltip mit "-1 to disable" und daher wollte ich den nutzen und der Fehler tauchte auf. Für mich ist das grundsätzlich kein Problem mit der 1 statt der -1, aber dachte der Fehler wäre trotzdem interessant zu wissen,

TA2k commented 5 months ago

nochmal via GitHub version installieren und testen

Backes89 commented 5 months ago

image

Fehler ist mal weg. Allerdings hast du durch das funktionierende if nun den return in 3519 welcher jetzt komplett die Wallbox Funktionalität abstellt statt nur die Charging-Records weil er komplett aus der Funktion "getElliData" rausgeht. . Jetzt kommt garnichts mehr an Daten rein. Oder vertu ich mich? Müsste die Funktion nicht auf zwei Funktionen aufgedröselt werden und der Teil mit den records ausgelagert werden?

Backes89 commented 5 months ago

image

Das Limit könnte doch sonst einzeln den Aufruf für die Records als Alternative noch unterbinden wenn nen If mit -1 als Vergleich vorangestellt wird.

TA2k commented 5 months ago

es war o gedacht die ganze Wallbox Funktionalität zu deaktivieren weil die vw box nur mit dem vw id account geht für skodapower wäre es sinnvoll den Eintrag auf 1 zu setzen

Backes89 commented 5 months ago

Achso. Sieht das Backend von VW da nochmal anders aus als bei Skoda?

TA2k commented 5 months ago

das nicht aber man kann sich nur mit einem vw id account einloggen

Backes89 commented 5 months ago

Rufe ich normalerweise mit nem ID Account sowohl Daten des Autos und die der Wallbox gleichzeitig ab? Bei Skoda muss ich ja einerseits ne Instanz für "Skoda Enyaq" für das Auto anlegen und "Skoda Powerpass" für die Wallbox. Bin da jetzt irgendwie verwirrt.

TA2k commented 5 months ago

Korrekt das ist der Unterschied.

Backes89 commented 5 months ago

Alles klar. Dann sollte man den Tooltip vielleicht abändern, dass die Option nur für VW nötig ist.