-
Notifications
You must be signed in to change notification settings - Fork 101
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
All the changes #59
base: master
Are you sure you want to change the base?
All the changes #59
Conversation
fix validity check to avoid no sleep condition
Added vehicle_is_charging and vehicle_is_driving methods
Renamed Functions and added usage to drive checks
Fix lephisto#51 Invalid Field Format
fix validity check to avoid no sleep condition
Added vehicle_is_charging and vehicle_is_driving methods
Added offline check to reduce polling.
Patch to prevent crashes when hostname lookups fail
Using best practices for cached layers, removed unneeded dependencies, and simply copied in local files (rather than pulling the project) which improves DX, speeds build, and fixes caching problems.
If you have multiple cars, you can now easily run multiple scrapers.
refactor(docker): use data/ rather than rely on /opt
Refactor/docker compose
Note is in the install script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not got my car yet, so can't test this. Though noticed a few things when reviewing the README
```bash | ||
# Enter your email and password to your tesla.com account | ||
teslaEmail="" | ||
teslaPassword="" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing git clone (and possibly swapping branch)?
git clone https://github.com/lephisto/tesla-apiscraper.git | |
cd tesla-apiscraper | |
# Swap to the latest stable, version, such as: | |
git checkout v2019.5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
Co-Authored-By: John R Barker <[email protected]>
I will test this next days and merge if everything fine.. |
Any updates on the merge? I’m waiting in the wings for the green light to pull it and use it :) Thanks @lephisto for everything! |
Sorry, I was out for some business, I will dig into this next days. promise :) |
We have an updated version of the scraper running on Lunars repo, with many more changes to come
https://github.com/lunars/tesla-apiscraper