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: fix grid_status #466

Merged
merged 1 commit into from
Apr 4, 2024
Merged

fix: fix grid_status #466

merged 1 commit into from
Apr 4, 2024

Conversation

ericdegroot
Copy link
Contributor

I recently had a Powerwall 3 installed, and I have no local API access. I'm using Tesla Custom Integration to integrate the battery into my Home Assistant setup via the owner cloud. The grid status binary sensor provided by that integration always inaccurately showed as 'Clear' or off. I traced this back to the PowerwallSite.grid_status property in this project. Changing this property to lookup the value in self._site_data instead of self._site_summary fixed the issue for me. Your test mock data suggests that site data was always the correct source, but I do not have access to any other powerwalls to see if this is unique to Powerwall 3 cloud access.

@ericdegroot ericdegroot changed the title PowerwallSite.grid_status tries to pull grid_status from _site_summary instead of _site_data fix: PowerwallSite.grid_status tries to pull grid_status from _site_summary instead of _site_data Apr 4, 2024
@alandtse
Copy link
Collaborator

alandtse commented Apr 4, 2024

Thanks!

@alandtse alandtse changed the title fix: PowerwallSite.grid_status tries to pull grid_status from _site_summary instead of _site_data fix: fix grid_status Apr 4, 2024
@alandtse alandtse merged commit 2c5e434 into zabuldon:dev Apr 4, 2024
4 of 5 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