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

Adds method to hit the solar 'history' API endpoint. #306

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

ArthurD
Copy link

@ArthurD ArthurD commented Jan 31, 2022

Very basic PR that is largely a copy/paste from the existing solar method/API call directly above it. This allows you to pull the 'solar history' for a given energy site (energy_site_id).

I used args instead of adding to the existing options object to avoid making any 'obtrusive' changes to existing code.

I have this in 'production' use and it's working well.

FYI / for potential documentation purposes, etc -- the new API call currently returns 2 days worth of day ("today" and "yesterday"), with the following being an example of the JSON returned:

JSON Results from API Endpoint:
https://gist.github.com/ArthurD/f688719e406a00db40f87a81f0592f87

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