Skip to content

Conversation

@core-hacked
Copy link
Contributor

@core-hacked core-hacked commented May 30, 2023

Changes:

  • Update ownerapi_endpoints.json on par with App v4.21.0-1717.
  • 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.

Owner API endpoint notes: Just BFF endpoints. No breaking changes or new endpoints on ownerapi url.

core-hacked and others added 2 commits May 30, 2023 19:19
Document changes in app behaviour/new way of requesting what the `data_request` endpoints had.
@core-hacked core-hacked changed the title Update ownerapi_endpoints.json to v4.21.0-1717 🔗 Update ownerapi_endpoints.json to v4.21.0-1717 & Update Docs 🔗📚 May 30, 2023
@timdorr
Copy link
Owner

timdorr commented May 31, 2023

I cherry-picked the endpoints updates for now. I'm going to mull over these larger changes for a sec. I just want to be sure we've got things organized in the right way, so it's not overwhelming or confusing for new users.

@core-hacked
Copy link
Contributor Author

I cherry-picked the endpoints updates for now. I'm going to mull over these larger changes for a sec. I just want to be sure we've got things organized in the right way, so it's not overwhelming or confusing for new users.

When I get the time I will move the doc changes to a separate PR so there are no unrelated/already merged commits. For now this can stay open. I will make a new PR once the above is done.

Kind Regards!

@core-hacked
Copy link
Contributor Author

Doc changes now in #725

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.

2 participants