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

Public API - Released talks: Add filter for master encoding tickets #185

Closed
saerdnaer opened this issue Sep 16, 2018 · 5 comments
Closed

Comments

@saerdnaer
Copy link
Collaborator

I think we should filter the output of https://github.com/crs-tools/tracker/blob/master/src/Application/Controller/API.php#L39 to only output master encoding tickets by default or by optional URL parameter

@a-tze
Copy link
Collaborator

a-tze commented Sep 16, 2018

Why not just group the output by fahrplan_id on the consumer side?

@jjeising
Copy link
Member

What are the current known consumers of this API? Why do they need filtering and which need all releases?

@saerdnaer
Copy link
Collaborator Author

The typical use case is to get a mapping of the talk id to the corresponding youtube urls, which is empty for the sub-formats.

@jjeising
Copy link
Member

I currently tend to a wontfix in this versions of the API. I would like to extend the API and provide a more helpful structure (e.g. formats grouped by ticket). This new API may also include filtering, but probably not, as there are currently not many consumers.

@saerdnaer
Copy link
Collaborator Author

Okay, created a new issue to discuss a new api #196 and closing this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants