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.

Sign up for GitHub

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

Jump to bottom

Added Anilist error log when the record exists on mapping but does not enrich with Anilist id #2728

Merged
merged 1 commit into from
Oct 20, 2024
Merged

Added Anilist error log when the record exists on mapping but does not enrich with Anilist id #2728

merged 1 commit into from
Oct 20, 2024

Conversation

ThisIsntTheWay
Copy link
Contributor

This change prevents the refiner from throwing an error in situations where matched entries in the reference list don't have an anilist_id attribute, as it is the case with this entry (at time of writing):

> $list | ? imdb_id -eq "tt3138698"

thetvdb_id    : 102261
imdb_id       : tt3138698
anisearch_id  : 6334
themoviedb_id : 357786
anidb_id      : 8357
type          : MOVIE

@anderson-oki anderson-oki changed the title AniList refiner: Gracefully handle missing anilist_id attribute Added anilist error log when the record exists on mapping but does not enrich with anilist id Oct 20, 2024
@anderson-oki anderson-oki changed the title Added anilist error log when the record exists on mapping but does not enrich with anilist id Added Anilist error log when the record exists on mapping but does not enrich with Anilist id Oct 20, 2024
@morpheus65535 morpheus65535 merged commit a9243c6 into morpheus65535:development Oct 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anderson-oki anderson-oki anderson-oki approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ThisIsntTheWay @anderson-oki @morpheus65535