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

When users just installed meters, trying to fetch data from 1 month back would fail #128

Open
4 tasks done
GuyKh opened this issue Jun 24, 2024 · 2 comments
Open
4 tasks done
Assignees

Comments

@GuyKh
Copy link
Owner

GuyKh commented Jun 24, 2024

System Health details

N/A

Checklist

  • I have enabled debug logging for my installation.
  • I have filled out the issue template to the best of my ability.
  • This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
  • This issue is not a duplicate issue of any previous issues..

Describe the issue

When users just installed meters, trying to fetch data from 1 month back would fail since the data is more recent.

Reproduction steps

1.Install a new smart_meter
2.install the component without a month passing
3.Check for iec: in energy dashboard
...

Debug logs

N/A

Diagnostics dump

No response

@GuyKh GuyKh self-assigned this Jun 24, 2024
@GuyKh
Copy link
Owner Author

GuyKh commented Jun 24, 2024

Per @maorcc report

@GuyKh
Copy link
Owner Author

GuyKh commented Jun 24, 2024

One optional solution would be to have the time to fetch if no recent statistics to be customizable in the config_flow rather than hardcoded 4 weeks (https://github.com/GuyKh/iec-custom-component/blob/main/custom_components/iec/coordinator.py#L355)

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

No branches or pull requests

1 participant