Skip to content

fix(tvdb): respect display language when fetching metadata#1889

Merged
fallenbagel merged 8 commits intoseerr-team:developfrom
TOomaAh:fix/tvdb-display-language-ignored
Sep 8, 2025
Merged

fix(tvdb): respect display language when fetching metadata#1889
fallenbagel merged 8 commits intoseerr-team:developfrom
TOomaAh:fix/tvdb-display-language-ignored

Conversation

@TOomaAh
Copy link
Copy Markdown
Contributor

@TOomaAh TOomaAh commented Sep 3, 2025

Description

Screenshot (if UI-related)

image image

To-Dos

  • Successful build pnpm build
  • Translation keys pnpm i18n:extract
  • Database migration (if required)

Issues Fixed or Closed

I verified the translation mapping by using the file 'src/context/LanguageContext.tsx' as a source for TMDB and mapping each language to those provided by TVDB by calling the route 'https://api4.thetvdb.com/v4/languages' (I can provide the call result).
The translation retrieval is working. When doing a partial translation of a series, no name is retrieved for the episode that is not translated.

I've re-added the line that was accidentally deleted, which was indeed preventing language retrieval.

To fetch TVDB translations, I need to call an API that returns paginated results. This API returns pages of 500 elements. To limit API calls somewhat, when the requested language matches the series' language, I retrieve all seasons corresponding to the series to reduce the number of calls.

For translation retrieval, I've added a fallback language (English). If neither the requested language nor the fallback language is available, the original translation (from the series' country of origin) is retrieved.

@TOomaAh TOomaAh marked this pull request as ready for review September 4, 2025 18:50
@github-actions github-actions bot added the merge conflict Cannot merge due to merge conflicts label Sep 8, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 8, 2025

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@TOomaAh TOomaAh force-pushed the fix/tvdb-display-language-ignored branch from a5535a3 to 86b3b87 Compare September 8, 2025 08:55
@github-actions github-actions bot removed the merge conflict Cannot merge due to merge conflicts label Sep 8, 2025
Copy link
Copy Markdown
Member

@gauthier-th gauthier-th left a comment

Choose a reason for hiding this comment

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

LGTM

@fallenbagel fallenbagel merged commit 4878722 into seerr-team:develop Sep 8, 2025
8 checks passed
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.

Display language not being taken into account with TVDB metadata provider

4 participants