Fix Trakt Authorization dialog on Trakt.tv

[i-a-c]

Fix Trakt Authorization dialog on Trakt.tv

[I-A-C]

That does look a little sketchy.. I'll have to look into that.

[drinfernoo]

@I-A-C It's very easym You can edit it directly on Trakt.tv, and it updates immediately.

[I-A-C]

I didn't have authority over the current Trakt API key, so I created a new trakt API key with icon and named ExodusRedux. If I upload a new Trakt API key, will that adversely affect the current installed user base or will it be as easy as reauthorizing?

[doko-desuka]

If the user has a cached token from a previous use, it'll be for the old API key.
When the user tries to go into their Trakt manager, it'll try to use this cached token but with the new key.
It'll keep failing in an obscure way, because the code sees that there's a cached token and assumes that it should definitely work with the key and will keep trying to do that.
It doesn't have a failsafe at the moment.

[drinfernoo]

In other words, users will have to reauthorize.

[doko-desuka]

Did anyone test it with the new API key? What are you getting?

[drinfernoo]

Is there a new API key? I only update whenever it hits the repo.

[doko-desuka]

I think so, there's a commit from Nov 15 on script.module.exodusredux:
I-A-C/script.module.exodusredux@288de83

[drinfernoo]

The changes went live in the repo today.