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

All the changes #59

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

All the changes #59

wants to merge 50 commits into from

Conversation

Geczy
Copy link

@Geczy Geczy commented Jul 23, 2019

We have an updated version of the scraper running on Lunars repo, with many more changes to come

https://github.com/lunars/tesla-apiscraper

ltfiend and others added 30 commits April 22, 2019 21:48
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 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
Copy link

@gundalow gundalow left a 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=""

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)?

Suggested change
git clone https://github.com/lephisto/tesla-apiscraper.git
cd tesla-apiscraper
# Swap to the latest stable, version, such as:
git checkout v2019.5

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed!

README.md Outdated Show resolved Hide resolved
@lephisto
Copy link
Owner

I will test this next days and merge if everything fine..

@mamiller93
Copy link

mamiller93 commented Aug 20, 2019

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!

@Geczy
Copy link
Author

Geczy commented Aug 25, 2019

@lephisto

@lephisto
Copy link
Owner

Sorry, I was out for some business, I will dig into this next days. promise :)

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.

8 participants