-
Notifications
You must be signed in to change notification settings - Fork 25
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Change API to be async * Modify docs * After looking at the Home Assistant integration that uses this library, I shouldn't have removed the logic to automatically create an http client session and the close method. Adds this functionality back. * Fix integration test * Based on review feedback, remove dependency on requests and misc review feedback. * Clean up closing logic and add tests to match * Update integration test to validate authentication cookie * Allow "unsafe" cookies in the integration test to allow the use of ip addresses instead of DNS names for the POWERWALL_IP environment variable * Move cookie fix to the integration test * Address review feedback with typing and make default http session created with "unsafe" cookies * Remove tox linting and type checking --------- Co-authored-by: Josh <[email protected]>
- Loading branch information
1 parent
b6a5f10
commit aa75e6f
Showing
10 changed files
with
620 additions
and
487 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -125,3 +125,6 @@ dmypy.json | |
.js | ||
.vscode | ||
.direnv | ||
|
||
# MacOS finder stuff | ||
.DS_Store |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.