Skip to content

Adds search by TVmaze ID feature and updates NBL site#324

Open
Audionut wants to merge 3 commits intoPurfview:masterfrom
Audionut:tvmaze-nbl
Open

Adds search by TVmaze ID feature and updates NBL site#324
Audionut wants to merge 3 commits intoPurfview:masterfrom
Audionut:tvmaze-nbl

Conversation

@Audionut
Copy link
Contributor

No description provided.

@Purfview
Copy link
Owner

Nice addition.
Can you update the doc on the main page?

@Purfview Purfview changed the title add tvmaze fetching and update nbl search Adds search by tvmazeid feature and updates NBL site Feb 27, 2026
@Purfview Purfview changed the title Adds search by tvmazeid feature and updates NBL site Adds search by TVmaze ID feature and updates NBL site Feb 27, 2026
@Audionut
Copy link
Contributor Author

I think TVDB is more accurate for fetching tvmaze, but that would require more work. Another solution, might be to try imdb, then tvdb, to fetch the tvmaze id, but that's quite heavy lifting.

Perhaps see how users fare, and any issues moving forward with missing ids, then I can tackle at that stage.

@Purfview
Copy link
Owner

I think TVDB is more accurate for fetching tvmaze

That would be with assumption that IMDb ID conversion to TVDb ID is perfect.

@Purfview
Copy link
Owner

Doesn't NBL support TVDb IDs?

@Audionut
Copy link
Contributor Author

Audionut commented Feb 27, 2026

No, they are tvmaze only.

That would be with assumption

Indeed. Perhaps the most robust option is to have %search_string% type fallback, but that would require fallback urls being added to site sections, and then the code to handle the fallback. I'm not sure it's worth it at this stage, but if I notice issues with NBL missing links moving forward, because tvmaze was not captured, I would be happy to tackle then.

The function would need to handle all missing types (from bad TVDB fetching also, for example), so that the fallback option could work script wide.

@Purfview
Copy link
Owner

Purfview commented Feb 27, 2026

Or maybe just have additional site config with %search_string%, like BTN has multiple.

The function would need to handle all missing types (from bad TVDB fetching also, for example), so that the fallback option could work script wide.

I thought about such feature before. I think there are some problems with the concept and other functionalities of the script.

@Audionut
Copy link
Contributor Author

Added title search back as an option.

I think there are some problems with the concept and other functionalities of the script.

Ok thanks. I may dabble at some stage in the far future.

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.

2 participants