Skip to content

feat: add anidb support to jellyfin scanner#1533

Merged
fallenbagel merged 5 commits into
seerr-team:developfrom
ale183:feature-jellyfin-anidb
Sep 25, 2025
Merged

feat: add anidb support to jellyfin scanner#1533
fallenbagel merged 5 commits into
seerr-team:developfrom
ale183:feature-jellyfin-anidb

Conversation

@ale183

@ale183 ale183 commented Mar 26, 2025

Copy link
Copy Markdown
Contributor

Description

This PR adds AniDB support in the Jellyfin scanner.

The AniDB ID is used only if there's no other provider id available to ensure it doesn't change the way it works now with TMDB/TVDB.

In some cases it doesn't work correctly, for example if the show is splitted in multiple season while on anidb it's a single season.
I know it's not optimal, but these are quite rare cases and i think it's better than nothing.

Screenshot (if UI-related)

To-Dos

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

@gauthier-th gauthier-th left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you please provide a few examples of shows where these changes improve the scan?

Comment thread src/i18n/locale/en.json
Comment thread server/lib/scanners/jellyfin/index.ts Outdated
@ale183

ale183 commented Apr 2, 2025

Copy link
Copy Markdown
Contributor Author

Could you please provide a few examples of shows where these changes improve the scan?

All anime shows can be used as an example. If someone uses only AniDB as a metadata provider, they won't receive any results from the scan. These changes are necessary to properly map AniDB shows to the corresponding TVDB/TMDB/IMDB show

@ale183 ale183 requested a review from gauthier-th April 13, 2025 01:40
Comment thread server/lib/scanners/jellyfin/index.ts Outdated
fallenbagel
fallenbagel previously approved these changes Sep 18, 2025

@fallenbagel fallenbagel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM
@gauthier-th Please review

Comment thread server/lib/scanners/jellyfin/index.ts Outdated
Co-authored-by: Gauthier <mail@gauthierth.fr>
gauthier-th
gauthier-th previously approved these changes Sep 25, 2025

@gauthier-th gauthier-th left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@fallenbagel

Copy link
Copy Markdown
Member

Please fix the build issues
@ale183

@fallenbagel fallenbagel merged commit b209e7f into seerr-team:develop Sep 25, 2025
9 checks passed
@gauthier-th

Copy link
Copy Markdown
Member

@all-contributors please add @ale183 for code

@allcontributors

Copy link
Copy Markdown
Contributor

@gauthier-th

I've put up a pull request to add @ale183! 🎉

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