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.

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

Changing to use new JSON api #57

Closed
ConorNevin opened this issue Nov 20, 2016 · 6 comments
Closed

Changing to use new JSON api #57

ConorNevin opened this issue Nov 20, 2016 · 6 comments

Comments

@ConorNevin
Copy link

Are you planning on changing to use the new JSON api given that they are deprecating the XML api per http://www.thetvdb.com/wiki/index.php/Programmers_API?

@lozbrown
Copy link

API deprecation date is looming "We are planning to discontinue the old API as of October 1st"

https://api.thetvdb.com/swagger

@jefflaing
Copy link

For the record, it looks like you are now official broken.

t = tvdb_api.Tvdb()
show = t['American Gods']
Traceback (most recent call last):
File "", line 1, in
File "build/bdist.macosx-10.10-intel/egg/tvdb_api.py", line 952, in getitem
File "build/bdist.macosx-10.10-intel/egg/tvdb_api.py", line 937, in _nameToSid
File "build/bdist.macosx-10.10-intel/egg/tvdb_api.py", line 884, in _getShowData
File "build/bdist.macosx-10.10-intel/egg/tvdb_api.py", line 650, in _getetsrc
tvdb_exceptions.tvdb_error: There was an error with the XML retrieved from thetvdb.com:
no element found: line 216, column 23
First try emptying the cache folder at..
/var/folders/yh/p0282tlx7wd_s5sbntc3667c0000gp/T/tvdb_api-u502
If this does not resolve the issue, please try again later. If the error persists, report a bug on
http://dbr.lighthouseapp.com/projects/13342-tvdb_api/overview

@thezoggy
Copy link

http://thetvdb.com/?string=american+gods&tab=listseries&function=Search
the entry with no title is prob the cause

@jefflaing
Copy link

Irrespective, they are serving bad XML for many series, not just that one, and that's the interface that they don't support, and no-one should expect they will fix it in a timely fashion.

@jankatins
Copy link
Contributor

There is a partial fork: https://github.com/dpimenov/tvdb_api/tree/newapi Unfortuinately, not all functions are ported yet and it breaks all users because the API is not translated (episodenumber is now airedEpisodeNumber)...

@dbr
Copy link
Owner

dbr commented Sep 16, 2017

tvdb_api==2.0 released which uses the new API thanks to everyone in #58 and #60

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

No branches or pull requests

6 participants