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: Fix refresh for old bluelink API #13785

Merged
merged 1 commit into from
May 7, 2024

Conversation

runtologist
Copy link
Contributor

@runtologist runtologist commented May 6, 2024

Fix #13772

Copy link
Contributor Author

@runtologist runtologist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#13713 uses StatusPartial instead of StatusLatest for statusG

@runtologist runtologist force-pushed the fix/bluelinkOldRefresh branch from 99b64fa to 0f68a4c Compare May 6, 2024 18:20
@andig
Copy link
Member

andig commented May 7, 2024

D.h. Fehler gefunden? Können wir das überprüfen?

@runtologist
Copy link
Contributor Author

Ich kann die alte API nicht testen mangels entsprechendem Fahrzeug. Vielleicht kann @steirerman testen? Alternativ bräuchte ich entsprechende Bluelink credentials von irgendwem.

@steirerman
Copy link

Ich kann die alte API nicht testen mangels entsprechendem Fahrzeug. Vielleicht kann @steirerman testen? Alternativ bräuchte ich entsprechende Bluelink credentials von irgendwem.

Ich könnte das machen, allerdings habe ich keine Ahnung wie ich das machen soll, da bräuchte ich etwas Unterstützung ;)

@steirerman
Copy link

steirerman commented May 7, 2024

@runtologist Was versteht man unter Bluelink credentials, könnte ich zur verfügung stellen.

@andig andig added the bug Something isn't working label May 7, 2024
@runtologist
Copy link
Contributor Author

runtologist commented May 7, 2024

@runtologist Was versteht man unter Bluelink credentials, könnte ich zur verfügung stellen.

user, password, vin.

Musst mir dann vertrauen, dass ich keine komischen Dinge mit Deinem Auto mache. ;) Wenn das für Dich passt, schlage ich vor Du generierst einen Einmal-Link mit den Informationen, z.B. mittels https://onetimesecret.com/, und schickst mir dann eine Mail an xxxxxx.

@runtologist
Copy link
Contributor Author

@andig Ich kann @steirerman's Problem auf master reproduzieren und auf diesem PR funktioniert's und zeigt die korrekten Daten. Besten Dank @steirerman für die Mithilfe.

@andig andig merged commit 1ff1513 into evcc-io:master May 7, 2024
7 checks passed
@VolkerK62
Copy link
Contributor

@runtologist reine Neugier, weil ich keine Probleme habe. Spielt dabei die VIN ein Rolle?

@runtologist
Copy link
Contributor Author

@VolkerK62 die VIN nutzen wir gar nicht in dem API Call, der fehlschlägt. Ich habe ehrlich gesagt keine Ahnung woran genau es liegt, ich habe nur festgestellt, dass sich eine kleine Änderung in der Logik eingeschlichen hat. Es gibt zwei Routen, die überlappende Informationen liefern. Wir haben zwischendurch "die andere" verwendet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hyundai Ioniq5 wird nicht mehr erkannt
4 participants