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

Document Changes in App behaviour / data requests | REL: #724 #725

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

core-hacked
Copy link
Contributor

Changes:

  • Document the new way of requesting what the data_request endpoints returned.
  • Remove obsolete endpoint from docs/summary
  • Update summary to feature new endpoints/states
  • Update some responses on par with the above changes.

Notes: Two unknown states have been added/used by the app. closures_state and location_state.
Location State might change if the car is in motion, someone needs to check that, I haven't had the time to check yet. Maybe Car/Config related. What closures state is, I have no idea.

Relates to #724

Comment on lines +120 to +122
"active_route_latitude": 0.0,
"active_route_longitude": 0.0,
"active_route_traffic_minutes_delay": 0.0,
Copy link

@longzheng longzheng Jun 16, 2023

Choose a reason for hiding this comment

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

This can actually return more data if a destination is set in the navigation

            "active_route_destination": "Home",
            "active_route_energy_at_arrival": 73,
            "active_route_latitude": -1.123456,
            "active_route_longitude": 1.123456,
            "active_route_miles_to_arrival": 6.485299,
            "active_route_minutes_to_arrival": 23.466667,
            "active_route_traffic_minutes_delay": 0.0,

@LexNastin
Copy link
Contributor

By the way: in the bluetooth api, closure are the following: doors, frunk/trunk, and charging port. They seem to just to bring that over to web (or if it was already there, perhaps just separating it out).

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.

3 participants