-
Notifications
You must be signed in to change notification settings - Fork 40
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
Setting charge mode started failing #1223
Comments
This is on 0.2.3 by the way. Also weird .. I had to revert |
I confirm that after having previously upgraded to 0.2.3 because of the hvacStatus change, I also just now had to revert int back to str - what the heck are they up to ?... |
Same here, and same for set charge mode. |
Yeah - it's infuriating!!! I'm happy to take a PR to revert the original change - but how long will it last? |
same here, though the error message, while similar, is slightly different
Using this script for Domoticz: https://github.com/HomeACcessoryKid/Domoticz-Renault-Plugin which uses this repo api |
@epenet how do you debug the endpoint changes locally? Do you have a rooted android/iOS device to hook into this? I don't mind digging deeper in the latest changes, just interested in how you keep up with their app (endpoint) changes. |
I no longer try to keep up. The worst part is that quite often even their app stops working!!! |
For reference I ended up downloading the My Renault android apk and uploading it to www.decompiler.com (https://www.decompiler.com/jar/591d0352c74840909291f718456bf807/base.apk), opened the decompiled code in Intellij CE. I did find the endpoint and it doesn't seem to have changed (if indeed this is the last apk). While debugging it seems that this endpoint just started working again for me. Is anyone else seeing this? @bizoo @HomeACcessoryKid @Zaphod-dev |
I still had issues, but just found the change in the json format. After fixing that, normal behaviour observed. Up till recent, there was different vocabulary for reading (res) the chargeMode and for changing (cmd) the chargeMode. so before:
and now
PS. this is from my python script referenced before |
Anyone willing to open a PR? |
I'm at a loss what has changed to be honest :-) I am not sure if this has always been this way but there is a difference between:
The first one will return Likewise when you set it to
I was gonna comment on the |
for me there is a new error, but if I do not call The key issue seems to be that MAYBE renault is using API versions, but we seem to tap into |
Side note: I have published a new release with a fix for |
About a day or two ago, the
/charge-mode
endpoint started failing.Retrieving still seems to work:
The text was updated successfully, but these errors were encountered: