Skip to content

Conversation

@Brett-S-OWB
Copy link
Contributor

Add manual SoC input to charge point card

@Brett-S-OWB
Copy link
Contributor Author

ich habe gerade die console.warn-Anweisung aus der Computer-Property im Store ChargePointConnectedVehicleSocManual entfernt, da das MQTT-Topic für ChargePointConnectedVehicleInfo nicht rechtzeitig geladen wurde. Das hat zu einem Fehler geführt, bei dem das Fahrzeug in der ChargePoint-Karte nicht ausgewählt werden konnte. Ich denke, der Warnhinweis ist ohnehin nicht nötig, da immer ein Standardfahrzeug mit dem ChargePoint verbunden ist.

@benderl
Copy link
Contributor

benderl commented Apr 11, 2025

Funktioniert bei mir. Bitte pass noch die Texte an. Die sind teilweise auf Englisch. ;-)
Dann darf das Symbol zum Bearbeiten nur angezeigt werden, wenn das Fahrzeug auch einen manuellen SoC verwendet!
Im Topic openWB/vehicle/X/soc_module/config muss dafür "type": "manual" gesetzt sein.

@LKuemmel LKuemmel force-pushed the feature_simplify_chargemode branch from 69e8a9b to 59578e4 Compare April 11, 2025 06:49
@Brett-S-OWB
Copy link
Contributor Author

Okay, jetzt funktioniert es auch bei mir. Vermutlich lag es daran, dass mein Backend nicht auf dem neuesten Stand war. Außerdem habe ich gemerkt, dass beim Neuladen oder wenn die Seite zum ersten Mal geladen wird, der Wert von manual soc nicht in den Eingabedialog (Input-Dialog) eingelesen wird, weil das Topic noch nicht abonniert ist. Ich habe das in einer async Funktion in der Komponente gelöst, bin mir aber nicht sicher, ob das die beste Lösung ist oder ob du einen anderen Vorschlag hast?

@benderl
Copy link
Contributor

benderl commented Apr 11, 2025

Ich habe den Code für das Eingabefeld überarbeitet. Wenn man konsequent auf Computed Properties setzt, sind auch keine async, watch oder andere "Tricks" notwendig.

@benderl benderl merged commit 9cc1cc2 into openWB:feature_simplify_chargemode Apr 11, 2025
1 check passed
@Brett-S-OWB Brett-S-OWB deleted the koala-web-theme branch April 24, 2025 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants