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

Add some documentation for REST API #3314

Closed
philippotto opened this issue Oct 4, 2018 · 3 comments · Fixed by #3376
Closed

Add some documentation for REST API #3314

philippotto opened this issue Oct 4, 2018 · 3 comments · Fixed by #3376
Assignees

Comments

@philippotto
Copy link
Member

Ideally, we should also make sure that very common use cases for the REST API are also covered via the UI (e.g., download all explorative tracings or similar).

@philippotto
Copy link
Member Author

  • Clarify high-level goal which makes REST API necessary.

@fm3
Copy link
Member

fm3 commented Oct 9, 2018

I’d say it is relatively safe to release those routes:

GET    /api/buildinfo

GET    /api/user
GET    /api/user/annotations

GET    /api/datasets

POST   /api/annotations/upload
POST   /api/annotations/Explorational/:id/duplicate
PATCH  /api/annotations/Explorational/:id/edit
PATCH  /api/annotations/Explorational/:id/finish
PATCH  /api/annotations/Explorational/:id/reopen
GET    /api/annotations/Explorational/:id/info
POST   /api/annotations/Explorational/:id/merge/Explorational/:mergedId
GET    /api/annotations/Explorational/:id/download

however, we should add a disclaimer note that these may change at any time (but the change will be documented). Only if this becomes a really important use case we should introduce api versioning etc.

@fm3
Copy link
Member

fm3 commented Oct 10, 2018

let’s wait for further feedback in discuss

@fm3 fm3 mentioned this issue Oct 18, 2018
2 tasks
@fm3 fm3 closed this as completed in #3376 Oct 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants