jerrellmardis/Amphitheatre

Switch from TMDb to TheTVDB for TV show information fetching

jerrellmardis opened this issue · 8 comments

The TheTVDB has a wider range of available shows so we should probably use that instead.

You might want to leave it as an option. TMDb's TV database is getting a lot better and it supports high resolution episode photos, which TheTVDb doesn't.

Perhaps, a settings to allow users to switch which service( TMDb vs TheTVDB ) to fetch shows from? Stating the pros and cons with each service?

Yeah, it should be fairly simple to do. Just create a "service" interface, which includes the basics (search, TV show retrieval, etc.) and create a class for each service that implements the interface and has its own code to search and such.

Exactly my thinking. Way to go.

Settings are terrible! How do I know which I should be using? The app
should do the right thing without the user having to make a decision on
something for which they have zero context on.
On Aug 13, 2014 3:40 AM, "Henry Addo" notifications@github.com wrote:

Exactly my thinking. Way to go.


Reply to this email directly or view it on GitHub
#31 (comment)
.

In this case there's no such thing as "the right thing". None of them are perfect.

TheTVDb's API is seriously outdated compared to TMDb. TMDb offers more information about TV shows and higher resolution images. On the other hand, the content on TheTVDb gets updated slightly faster due to a larger user group, which might appeal to some users.

So we use both. One for images and overview and the other for per-episode
data.

On Wed, Aug 13, 2014 at 8:33 AM, Michell Bak notifications@github.com
wrote:

In this case there's no such thing as "the right thing". TheTVDb's API is
seriously outdated and TMDb offers more information about TV shows and
higher resolution images. On the other hand, the content on TheTVDb gets
updated slightly faster, which might appeal to some users.


Reply to this email directly or view it on GitHub
#31 (comment)
.

So they both fallback on each other... Makes sense