-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
Remove localhost reference to fix encryption
Requires trakt API key, uses cached tvrage ID's provided by trakt
I'm not sure why this has conflicts, might be because of my css fix from last year. |
Yeh small change in style.css |
Thanks for this! However, I was getting this error when I try to start it.
So I added an indented block at line 280. But now I am getting this error. I know almost nothing about python, so I am stuck here.
|
Those are indentation errors, check the spacings / tabs |
From lines 280-287, I deleted the spaces before the code and replaced it with indentations using the tab key. After that, it started up fine. |
Sorry bout the indentation errors. Github editor is somewhat lacking. On Wed, 16 Sep 2015 2:59 am judhat2 notifications@github.com wrote:
|
Fumbled a bit, so it doesn't show up as merged on the pull request - but changes are merged. |
This is a crude fix to get tvrage ID's working again until a proper solution is found. Unfortunately I don't know python so this was just a lot of googling.
I found the trakt API the best solution to use as it has a lot of room to expand and should let us fix the "trending" module. Here's the API docs: http://docs.trakt.apiary.io/#
I've left my API key out, it should probably be a config item. see: trakt-api-key (https://trakt.tv/oauth/applications/new)
I believe this is a temporary fix as any new TV shows that trakt hasn't cached will probably still error. Needs full transition to trakt or tvdb.