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

feat: add cooling seats #470

Merged
merged 6 commits into from
Jun 6, 2024
Merged

feat: add cooling seats #470

merged 6 commits into from
Jun 6, 2024

Conversation

dandelionclock
Copy link
Contributor

@dandelionclock dandelionclock commented Jun 3, 2024

Issue 977 under Tesla Custom Integration

Added ability to retrieve states of and change state of cooled seats

Tesla seems to have changed the API call for REMOTE_SEAT_HEATER_REQUEST without documenting the change. Instead of the parameter "heater", it now wants "seat_position"

Auto seat climate still doesn't work. It now requires parameters "auto_seat_position" as well as "seat_position", and it's saying my car doesn't support this command. Maybe it can only be used on cars with both heated and ventilated seats?
@dandelionclock dandelionclock changed the title fix: heated seats (Issue 957) fix: heated seats (Issue 957), feat: add cooling seats (Issue 977) Jun 4, 2024
@dandelionclock dandelionclock marked this pull request as draft June 4, 2024 14:38
@dandelionclock dandelionclock marked this pull request as ready for review June 5, 2024 05:05
@alandtse
Copy link
Collaborator

alandtse commented Jun 5, 2024

Please don't mix and match PRs. Use separate branches for different things.

@dandelionclock dandelionclock changed the title fix: heated seats (Issue 957), feat: add cooling seats (Issue 977) feat: add cooling seats (Issue 977) Jun 6, 2024
@dandelionclock
Copy link
Contributor Author

done thanks

@alandtse alandtse changed the title feat: add cooling seats (Issue 977) feat: add cooling seats Jun 6, 2024
@alandtse alandtse merged commit d4fe98b into zabuldon:dev Jun 6, 2024
6 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.

2 participants