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 grouped season fetching #36

Open
Naliath opened this issue Dec 27, 2013 · 2 comments
Open

Add grouped season fetching #36

Naliath opened this issue Dec 27, 2013 · 2 comments
Assignees
Milestone

Comments

@Naliath
Copy link
Collaborator

Naliath commented Dec 27, 2013

Based on the following comment in the forums we could add something that wraps this functionality.

Is it possible to fetch all episodes with a single query?

If I understand correctly, currently /3/tv/:id returns only the available season numbers, and I'd have to send a request for each season to /3/tv/:id/season/:num in order to fetch all episodes. This would be
undesirable in some cases, where there are just way too many seasons.

Thank you for your response in advance.

EDIT: Nevermind, I've figured it out. Solution in case anyone else is bothered by this: /3/tv/:id?api_key=:key&append_to_response=season/1,season/2[,etc]

@LordMike
Copy link
Collaborator

We can make special methods that wrap the functionality. Obviously, we can't do it with the current layout, as they're for single-objects only.

But it might be a really good thing to have though.

@LordMike LordMike modified the milestones: Release 0.9 (async), Undetermined Jun 18, 2015
@LordMike LordMike modified the milestones: Undetermined, Release 0.9 (async) Jul 30, 2015
@Naliath
Copy link
Collaborator Author

Naliath commented Jan 29, 2016

There is a maximum number of calls one can make and each of these chained calls counts to that limit. So if we decide to implement this we should take this into account and perhaps split the seasons over multiple calls.

@Naliath Naliath self-assigned this Mar 23, 2017
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

No branches or pull requests

2 participants