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

Patch generic apis #312

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

Conversation

GaPhi
Copy link
Contributor

@GaPhi GaPhi commented Feb 12, 2022

Fixes #311.

Changes proposed in this pull request:

  • Add low level Tesla get/post API that manage Authentication header
  • Add low level Tesla vehicleGet/vehiclePost API that manage Authentication header and vehicleID part of the URL
  • Add command parameters to all API calls with a "args" object
  • Add query string parameters to get/post API calls with a "qs" object

This could break a little the compatibility with previous version (then tag 4.11.0 proposed) but should not be the case anymore for future versions.

GaoMjun and others added 30 commits June 16, 2021 11:07
Consider username/password as access/refresh tokens when username does not include '@' (email address)
See https://tesla-api.timdorr.com/api-basics/vehicles#get-api-1-vehicles-id

Adding this and deploying it to apiary will unblock adding support for this endpoint
to TeslaJS
These functions use `options.vehicleID` to make a call to fetch a specific
Vehicle from the Tesla API, rather than fetching all vehicles and using
`carIndex` to select the requested vehicle.

Fixes mseminatore#273
Add sync & async calls to SetChargingAmps API
Add setScheduledCharging calls API
Add setScheduledDeparture API calls
- setChargingAmps
- setScheduledCharging
- setScheduledDeparture
One parameter was missing!
NPM package with @GaPhi scope
Scheduled times can be any value in minutes since midnight local time, even if Tesla apps do not propose it.
windowControl needs lat/lon for the 'close' command.
NB: Cannot test it on my car, maybe a GET request is required rather than POST?

Fix a typo too
This is probably a small copy paste mistake from steeringHeater function above.
Fix typo in maxDefrost function name
Add vehicleById and vehicleByIdAsync function
Add "/api/1/vehicles" to API blueprint
Redirect to registered region
Bumps [ws](https://github.com/websockets/ws) from 8.2.2 to 8.3.0.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@8.2.2...8.3.0)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
This argument is :
- true if (preconditioning_enabled or off_peak_charging_enabled)
- false otherwise

NB: this condition may change in the future
Bumps [ws](https://github.com/websockets/ws) from 8.2.2 to 8.3.0.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@8.2.2...8.3.0)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Copy link
Contributor Author

@GaPhi GaPhi left a comment

Choose a reason for hiding this comment

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

Missing tests or docs should be ok now

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.

Low level request API
4 participants