-
-
Notifications
You must be signed in to change notification settings - Fork 785
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
Conversation
There was a problem hiding this 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
99b64fa
to
0f68a4c
Compare
D.h. Fehler gefunden? Können wir das überprüfen? |
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 ;) |
@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. |
@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. |
@runtologist reine Neugier, weil ich keine Probleme habe. Spielt dabei die VIN ein Rolle? |
@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. |
Fix #13772