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

Twitch "kraken" API is no longer maintained. #6

Open
esanwit opened this issue Feb 27, 2022 · 5 comments
Open

Twitch "kraken" API is no longer maintained. #6

esanwit opened this issue Feb 27, 2022 · 5 comments

Comments

@esanwit
Copy link

esanwit commented Feb 27, 2022

The current Twitch provider depotwarehouse/oauth2-twitch is based on the now defunct Kraken API.

There is a new provider vertisan/oauth2-twitch-helix which should be drop in compatible.

Any chance of getting this included?

@KevinConsen
Copy link

I'm dealing with the same issue

@rhukster
Copy link
Member

Could you guys try the latest develop branch to see if that works? I don't have a setup for Twitch currently so you probably will know best.

@esanwit
Copy link
Author

esanwit commented Feb 28, 2022

@rhukster Seems it's not drop in compatible. Even on the same framework.

I had to update the TwitchProvider with the following:

    public function getUserData($user)
    {
        $data_user = [
            'id'         => $user->getId(),
            'login'      => $user->getLogin(),
            'fullname'   => $user->getDisplayName(),
            'email'      => $user->getEmail(),
            'twitch'      => [
                'avatar_url' => $user->getProfileImageUrl(),
                'bio' => $user->getDescription(),
                'type' => $user->getType()
            ]
        ];

        return $data_user;
    }

After that it appears to work

@esanwit
Copy link
Author

esanwit commented Mar 26, 2022

@rhukster Can you take a look at the above comment :-)

@rhukster
Copy link
Member

Pull request please. I have no way to test currently.

@rhukster rhukster reopened this Mar 26, 2022
rhukster added a commit that referenced this issue Mar 30, 2022
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

3 participants