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

TeslaFi awake_start and awake_stop now function correctly #936

Closed
wants to merge 5 commits into from

Conversation

jjermany
Copy link

This update fixes the awake_start and awake_stop scripts. It checks sentry state from TeslaFi directly. If sentry mode is already on then it leaves it on after the archive process. If it was not on, it will turn off. This ensures that user's who want sentry mode on in the first place will be left with it on. Otherwise sentry mode gets turned back off after the archive process.

Now we use TeslaFI telemetry data to gather the Sentry state. This code will ensure that we only turn on sentry if not already on and not turn off if sentry was already enabled.
Copy link
Author

@jjermany jjermany left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated awake_start and awake_stop

@miles267
Copy link

Thank you! I’m holding off on updating my Teslausb until @marcone has confirmed this was merged into the main branch.

@jjermany
Copy link
Author

Sweet, I can confirm that this code works after testing tonight.

This was referenced Feb 26, 2025
@jjermany jjermany closed this Feb 28, 2025
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