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

Remote control features #677

Merged
merged 2 commits into from
Nov 25, 2023
Merged

Remote control features #677

merged 2 commits into from
Nov 25, 2023

Conversation

papageigit
Copy link
Contributor

The same additions as the previous pull request, but now without changing RemoteClient.py :)

I did not get around to testing the horn (did not want to wake up the neighbors), please someone else test it and advise if the count actually works. I have heard that it only horns three times regardless of set count but cannot verify this.

The lights do work, but they always flash for 10 seconds, regardless of set duration in PSACC. Maybe there is a way to remove the duration field in views.py (so no editing of RemoteClient.py)? I have tried it but ended up with no results.

Lock and unlocking doors also work. They work just like preconditioning the car. So 1 to lock and 0 to unlock.

Added the above to the readme and also cleaned that file up a bit.

@Mikado8231
Copy link
Contributor

What is your car model ?

On my side, I try to test with my Peugeot e208 but it is not eligible

@papageigit
Copy link
Contributor Author

e-2008. Do you have the required services activated? (Used to be called "Remote Control" but I believe this is now part of "Connect Plus")

@papageigit
Copy link
Contributor Author

Please also post the error here so it can be included in the FAQ if necessary.

@Mikado8231
Copy link
Contributor

OK, I've not this option.
I'm only with an old service to have a minimum actions (charge and preconditioning)
https://services-store.peugeot.fr/store/e-commandes-distance-0

I can't help you to tests these new features :'(

@flobz
Copy link
Owner

flobz commented Nov 25, 2023

thanks lgtm !

@flobz flobz merged commit 50760eb into flobz:master Nov 25, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants