-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Enode API Changes #87
Open
AndrewLester
wants to merge
23
commits into
main
Choose a base branch
from
h4i/enode
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Commits on Apr 23, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 23e4aa8 - Browse repository at this point
Copy the full SHA 23e4aa8View commit details -
Configuration menu - View commit details
-
Copy full SHA for cd99e2e - Browse repository at this point
Copy the full SHA cd99e2eView commit details
Commits on Apr 24, 2023
-
Fix lint errors Finish both endpoints Fix formatting and lint errors Update pv_site_api/main.py Co-authored-by: Andrew Lester <[email protected]> Update pv_site_api/main.py Co-authored-by: Andrew Lester <[email protected]> Update pv_site_api/main.py Co-authored-by: Andrew Lester <[email protected]> Address PR comments Update pyproject.toml Fix lint errors Fix formatting Fix imports Fix bugs Remove unused import Fix formatting Fix import order Create enode_auth.py Define basic auth function Get api key from env Finish part 1
Configuration menu - View commit details
-
Copy full SHA for db19834 - Browse repository at this point
Copy the full SHA db19834View commit details -
Remove test end point clean up Remove print messages Make getenv set variable to str make format lint and format Remove unecessary check Remove test files Use env var for enode token url Change get_enode_access_token to return request Refactor get_enode_access_token() into EnodeAuth class change 404 to 401 format Remove test code Remove print Address PR changes Update enode constructor to take in client_id, secret, and url Add type hints Add inverters test for site specific Fix lint errors Modularize code Fix lint errors
Configuration menu - View commit details
-
Copy full SHA for 0eddcde - Browse repository at this point
Copy the full SHA 0eddcdeView commit details -
Add enode link URL endpoint (#86)
* Add enode link URL endpoint * Fix lint * Rename test var for clarity * Add comment to mock urls fixtuer * Use redirect response
Configuration menu - View commit details
-
Copy full SHA for e1dadaf - Browse repository at this point
Copy the full SHA e1dadafView commit details -
[WIP] Add put endpoint for editing site (#88)
* Bump version: 0.0.36 → 0.0.37 * TDD: add failing test, looking at forecasts in the future * add start utc filter on forecast future qery * lint * lint * isort * add freeze gun to dev dependencies * self PR comments * Bump version: 0.0.37 → 0.0.38 * Make site_uuid field on PVSiteMetadata model optional * Make pydantic field optional * Move test site with real UUID to real db test * Don't use uuid in tests * Add part of check back * Ensure autogenerated site uuid is not null in database * Add WHERE statement on query for past forecasts This doesn't change the result but speeds up the query in general. * Bump version: 0.0.38 → 0.0.39 * Bump version: 0.0.39 → 0.0.40 * add logging * lint * run blacks * add extra logs * Bump version: 0.0.40 → 0.0.41 * add structlogging * add structlog to requiremwnts * add logging * Bump version: 0.0.41 → 0.0.42 * Add structlog initialisation * blacks * isort * Bump version: 0.0.42 → 0.0.43 * Uniformize check to FAKE and fix related test * Set codecov targets * Hard-code the "now" time for all tests This means that the same tests are run now matter the time at which we run them. * add caching * fix * lint * isort * fix for routes calling routes * lint * Bump version: 0.0.43 → 0.0.44 * PR comment * add args to cache * lint * Update pv_site_api/cache.py Co-authored-by: Simon Lemieux <[email protected]> * tidy * Bump version: 0.0.44 → 0.0.45 * What I've got * Add basic authorization to the /sites* routes The caller of the routes must have a proper Bearer authorization header set. * refactor * add tests * fix error in import * update pvsite-datamodel (in line with main) * fix bug * fix incompatible types * lint and format * add site existence check * add back correct datamodel dependency * fix test name * allow tests to pass by adding fake condition * remove 404 check * use or instead of is not none * Bump version: 0.0.45 → 0.0.46 * add site existence check * format and lint * Bump version: 0.0.46 → 0.0.47 * add post endpoint * add test back in * fix errors and format * add site name to test * fix tiny error * fix another small error * run tests * fix bugs * lint and format * fix fake * :pleading-face: * yet another is_fake() addition * address comments --------- Co-authored-by: Peter Dudfield <[email protected]> Co-authored-by: BumpVersion Action <bumpversion@github-actions> Co-authored-by: peterdudfield <[email protected]> Co-authored-by: AndrewLester <[email protected]> Co-authored-by: Simon Lemieux <[email protected]> Co-authored-by: devsjc <[email protected]> Fix some functions Fix Enode link endpoint Remove duplicate fixture Rename /inverters to /enode/inverters
Configuration menu - View commit details
-
Copy full SHA for 3541f2f - Browse repository at this point
Copy the full SHA 3541f2fView commit details -
Add post inverters for site endpoint
Move a few things Fix pyproject Fix Fix tests Fix import order Default the token URL Add some tests for other states Fix last missing coverage in main.py
Configuration menu - View commit details
-
Copy full SHA for f491656 - Browse repository at this point
Copy the full SHA f491656View commit details -
Configuration menu - View commit details
-
Copy full SHA for f4258e3 - Browse repository at this point
Copy the full SHA f4258e3View commit details -
Configuration menu - View commit details
-
Copy full SHA for ec1898c - Browse repository at this point
Copy the full SHA ec1898cView commit details
Commits on Apr 25, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 6715206 - Browse repository at this point
Copy the full SHA 6715206View commit details
Commits on Apr 27, 2023
-
Configuration menu - View commit details
-
Copy full SHA for d904f3a - Browse repository at this point
Copy the full SHA d904f3aView commit details -
Configuration menu - View commit details
-
Copy full SHA for ab5a5c1 - Browse repository at this point
Copy the full SHA ab5a5c1View commit details -
Configuration menu - View commit details
-
Copy full SHA for e4c0922 - Browse repository at this point
Copy the full SHA e4c0922View commit details
Commits on Apr 28, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 94c3767 - Browse repository at this point
Copy the full SHA 94c3767View commit details
Commits on Apr 29, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 6329cb1 - Browse repository at this point
Copy the full SHA 6329cb1View commit details
Commits on May 4, 2023
-
Configuration menu - View commit details
-
Copy full SHA for af3ca34 - Browse repository at this point
Copy the full SHA af3ca34View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4a04fd5 - Browse repository at this point
Copy the full SHA 4a04fd5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 381f849 - Browse repository at this point
Copy the full SHA 381f849View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5d622a2 - Browse repository at this point
Copy the full SHA 5d622a2View commit details -
Configuration menu - View commit details
-
Copy full SHA for e185d93 - Browse repository at this point
Copy the full SHA e185d93View commit details
Commits on May 16, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 246724d - Browse repository at this point
Copy the full SHA 246724dView commit details
Commits on May 18, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 7e80ee4 - Browse repository at this point
Copy the full SHA 7e80ee4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 819f55f - Browse repository at this point
Copy the full SHA 819f55fView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.