Skip to content

Conversation

ZackBoe
Copy link
Contributor

@ZackBoe ZackBoe commented Jul 25, 2025

Initially created combined API but moved to separate TMDBSeriesAPI and TMDBMovieAPI because TMDB APIs include the type in the URL. Not super thrilled about this, but it works.

Using one TMDBAPI class for both series & movie:

  • searchByTitle is workable with search multi
  • Would need to modify queryDetailedInfoById and getById to include type.
  • 'Create Media DB entry by id' command won't work without getting the type somehow.

Notes:

  • TMDB "API Key" is used, not "API Read Access Token" - unsure if this is clear enough.
  • Trimmed casts to first 5
  • Unsure how to format StreamingServices but this is available. Requires attribution, however.
  • Add country, boxOffice and ageRating fields for movies and series #196: Country is available as origin_country, original_language, or production_countries
  • Using 780w images. original is available for larger images, but can impact performance in bases & dataview.

Series notes:

  • The series level runtime field is deprecated (in favor of per season/episode runtimes), but many shows still have this. Many series have multiple runtimes - just grabbing the first.
  • Using production_companies (MGM Television, Double Secret Productions) for studio but this could also be networks (Showtime, Syfy)
  • Including credits, which on TMDB is for the latest season. Could swap to aggregate_credits
  • Add country, boxOffice and ageRating fields for movies and series #196: Content Rating is available per location.
  • I'd like to have a field for next_air_date but I can see this being very quickly (weekly, even daily) outdated. I'll just track this manually for my own usecase.

Movie notes:

@ferrolox
Copy link

@mProjectsCode Hi, just started to use your plugin today and I love it, I was wondering however when the TMDB integration is planned to go into production since it has been idle in the TMDB branch for quite some time now?

Copy link
Owner

@mProjectsCode mProjectsCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add type definitions for the API responses, similar to what has been done for existing APIs.

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

Successfully merging this pull request may close these issues.

3 participants