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

Use append to response to minimize the amount of API requests #420

Closed
JVT038 opened this issue Jul 3, 2023 · 1 comment · Fixed by #431
Closed

Use append to response to minimize the amount of API requests #420

JVT038 opened this issue Jul 3, 2023 · 1 comment · Fixed by #431
Labels
enhancement New feature or request priority: middle A little bit important

Comments

@JVT038
Copy link
Collaborator

JVT038 commented Jul 3, 2023

Description

TMDB apparently has a feature called append to response which can combine multiple API requests into one.

In the case of Movary, when adding a movie, various requests are made.
One request for the movie details and another one for the movie credits.
These requests can be combined like this: https://api.themoviedb.org/3/movie/27205?append_to_response=credits&language=en-US, resulting in a response where the credits are automatically included.
We should refactor the API calls to use this as whenever possible, so we reduce the amount of API requests and reduce the loading times.

Desired Behavior

Use append to response instead of separate API requests

Additional Context

No response

@leepeuker leepeuker added enhancement New feature or request priority: middle A little bit important labels Jul 3, 2023
@leepeuker
Copy link
Owner

I found one case (movie details and credits endpoints) where we could utilize the append to response feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: middle A little bit important
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants