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.

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

Bluelink: use charge port open as indicator for connected status #7150

Merged
merged 1 commit into from Apr 1, 2023

Conversation

andig
Copy link
Member

@andig andig commented Apr 1, 2023

Fix #7139

@andig andig added the enhancement New feature or request label Apr 1, 2023
@andig andig merged commit 7226a38 into master Apr 1, 2023
5 checks passed
@andig andig deleted the fix/hyundai branch April 1, 2023 12:10
@VolkerK62
Copy link
Contributor

Sorry. Ich habe den geschlossenen Zustand nicht kontrolliert. Ich bin von 0 ausgegangen.
Aber, total fies, wenn die Klappe zu ist, ist der Status = 2.
Das funktioniert so also nicht. Besser wäre =1.

@andig
Copy link
Member Author

andig commented Apr 1, 2023

Ist geändert.

@VolkerK62
Copy link
Contributor

Soc:           78%
Capacity:      72.0kWh
Charge status: B
Range:         312km

Status "B" hatte ich bei meine Auto noch nie. :-)
Genial

@andig
Copy link
Member Author

andig commented Apr 1, 2023

Ein kleiner Schritt für die Menschheit :)

@tobiasbayer
Copy link
Contributor

Ich habe nicht ganz verstanden, welches Problem das hier löst.
Die prompte Anzeige des Autos als "verbunden" hat bei mir jedenfalls schon immer direkt nach dem Anstecken funktioniert.
Bei mir scheint dadurch allerdings ein anderes Problem aufgeworfen zu werden:
Nach dem Abstecken wird das Auto weiterhin als "verbunden" angezeigt. Wahrscheinlich, weil der Status ja nicht mehr direkt abgefragt wird und im Cache immer noch "Charge port open" steht.

@andig
Copy link
Member Author

andig commented Apr 13, 2023

@tobiasbayer bitte neues Issue. @tobiasbayer und @VolkerK62 bitte mal Euer JSON nebeneinander legen ob wir das auseinander halten können. Falls nein: revert.

@tobiasbayer
Copy link
Contributor

Fall gelöst: Bei mir klappt das, weil ich am Fahrzeug den RFID-Identifier hinterlegt habe. batteryPlugin ist bei mir auch 0.
Und das Problem mit der "verbunden"-Anzeige, obwohl abgesteckt, kann ich auch nicht mehr reproduzieren. Passt also alles, wie es ist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hyundai Ioniq 5 als seperates Fahrzeug aufnehmen
3 participants