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

[renault] Support for MyDacia accounts and Dacia Spring car #14227

Closed
raulmatei opened this issue Jan 15, 2023 · 5 comments · Fixed by #15101
Closed

[renault] Support for MyDacia accounts and Dacia Spring car #14227

raulmatei opened this issue Jan 15, 2023 · 5 comments · Fixed by #15101
Labels
enhancement An enhancement or new feature for an existing add-on

Comments

@raulmatei
Copy link

Your Environment

  • Version used: openHAB 3.4.0 / Renault Binding 3.4.0
  • Chrome 109.0.5414.74
  • openjdk version "11.0.16" 2022-07-19
    OpenJDK Runtime Environment (build 11.0.16+8-post-Debian-1deb11u1)
    OpenJDK 64-Bit Server VM (build 11.0.16+8-post-Debian-1deb11u1, mixed mode)
  • NodeJS v16.19.0
  • Operating System: Debian GNU/Linux 11 (bullseye)
    Kernel: Linux 5.15.76-v8+
    Architecture: arm64

I'm trying to get the Renault Binding to work with MyDacia account and I'm not able to because the account type is hard coded to MYRENAULT account type here:

if (accounts.get(i).getAsJsonObject().get("accountType").getAsString().equals("MYRENAULT")) {
.

Authentication using https://github.com/hacf-fr/renault-api works as expected and I'm able to query the Renault APIs to get the car details, battery-status, location, cockpit, hvac-status. The lock-status is not working, there are some limitations for Dacia Spring cars discussed here: hacf-fr/renault-api#534. I haven't tried the actions yet but I'm willing to do that soon.

I'm not a Java programmer, but I can help with testing if someone gets an update that allows MYDACIA accounts to get registered.

Regards,
Raul.

@raulmatei raulmatei added the enhancement An enhancement or new feature for an existing add-on label Jan 15, 2023
@dougculnane
Copy link
Contributor

dougculnane commented Mar 12, 2023

@raulmatei There was a recent PR #14527 that might be of interest to you. It adds the charge pause channel which might work on your car. You should be able to test this with the latest openhab 4.0-snapshot.

I can add a line to get the accountType=="MyDacia" if the accountType=="MYRENAULT" does not find anything. I have no way to test this so you would need to test it for me. What is the exact value I need to look for "MyDacia", "myDacia", "MYDACIA"...

@rouallet
Copy link

rouallet commented Jun 4, 2023

Dear Dougculname,

I just changed it to "MYDADIA", build and test it: working perfectly fine:
image

My test setup: openhab 3.4.4, on a rpi4.
A Dacia Spring.

Regards,

Robin

@dougculnane
Copy link
Contributor

@rouallet Thanks to the test.

This will be quite easy to add in a 2nd loop if no MyRenault account is found. I will prepare a PR for including in openHAB 4 as soon as I can.

@dougculnane
Copy link
Contributor

PR to close this issues here: #15101

@dougculnane
Copy link
Contributor

PR Merged (#15101) and so we can close this issue. Support for MyDacia accounts will be in OpenHAB 4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants