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

fix: Corrected remote start types #117

Merged
merged 1 commit into from
Nov 9, 2021
Merged

fix: Corrected remote start types #117

merged 1 commit into from
Nov 9, 2021

Conversation

cdnninja
Copy link
Collaborator

@cdnninja cdnninja commented Nov 9, 2021

No description provided.

@cdnninja cdnninja requested a review from fuatakgun November 9, 2021 18:43
@dahlb
Copy link
Contributor

dahlb commented Nov 9, 2021

seems it would be cleaner to move this logic into the CA implementation of start_climate to call it's ev method internally if engine is ev

@cdnninja
Copy link
Collaborator Author

cdnninja commented Nov 9, 2021

If so I need to pass engine type into the API as the API isn't aware of engine type. I actually did this in a previous version but removed that type of logic as I have to decode and store engine type in both vehicle and api. Or pass it into the start call. Open to suggestions.

@dahlb
Copy link
Contributor

dahlb commented Nov 9, 2021

I haven't used those functions yet but based on initially look, with this and some hacks to track current version of vehicle maybe these or at least some of the api fuctions would benefit from adding vehicle object to signatures

@cdnninja
Copy link
Collaborator Author

cdnninja commented Nov 9, 2021

I am merging this to get it fixed as a temp fix. @fuatakgun is going to put some time into how to refactor some of this. We are hitting issues with different APIs needing different things.

@cdnninja cdnninja merged commit 6ccbeb2 into master Nov 9, 2021
@cdnninja cdnninja deleted the cdnninja-patch-1 branch November 14, 2021 17:23
@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants