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 Today/Yesterday readings #228

Merged
merged 1 commit into from
Dec 3, 2024
Merged

fix: Fix Today/Yesterday readings #228

merged 1 commit into from
Dec 3, 2024

Conversation

GuyKh
Copy link
Owner

@GuyKh GuyKh commented Nov 26, 2024

addresses issues raised in #197

@GuyKh GuyKh force-pushed the issue-197 branch 2 times, most recently from 8925271 to c6c5fa1 Compare November 26, 2024 08:05
@tzagim tzagim mentioned this pull request Dec 1, 2024
4 tasks
@@ -508,19 +516,12 @@ async def _update_data(

# fallbacks for future consumption since IEC api is broken :/
if (
not future_consumption[device.device_number]
not future_consumption.get(device.device_number)
Copy link
Owner Author

Choose a reason for hiding this comment

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

This was an annoying bug...

@GuyKh GuyKh merged commit 8edc1b2 into main Dec 3, 2024
5 checks passed
@GuyKh GuyKh deleted the issue-197 branch December 3, 2024 12:05
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.

1 participant