search

blakejoy / tmdb-ts

Typescript client wrapper for TheMovieDB (TMDB) v3
MIT License
37 stars 12 forks source link

Allow pagination with Trending endpoint #45

Closed VictorienDruon closed 4 months ago

VictorienDruon commented 7 months ago

Hi, thank you for your efforts on this project.

I've noticed that it's currently not possible to pass a 'page' parameter with the Trending endpoint. This limitation hinders the use of pagination. Although it's not explicitly documented in the TMDB documentation, I've found that we can successfully pass a 'page' parameter in the URL with the trending endpoints, and it functions correctly. The current data returned also includes the necessary information to enable this functionality (e.g., total_pages, page, etc.).

Therefore would it be possible to add a 'page' parameter to the Trending endpoint?

Thanks in advance, and keep up the excellent work!

blakejoy commented 6 months ago

Yup that's definitely possible @VictorienDruon . I can work on getting it done or you can make a Pull Request if you would like?