Update GitHub provider to recognize verified email #703

Merged
merged 2 commits into from Dec 5, 2016

Projects

None yet

2 participants

@forevermatt
Contributor

I tried to follow the existing coding style. Fixes #702.

@forevermatt forevermatt Update GitHub provider to recognize verified email
I tried to follow the existing coding style. Fixes #702.
2c3d717
@forevermatt
Contributor

The GitHub OAuth documentation about users is at https://developer.github.com/v3/users/emails/

@forevermatt
Contributor

If you would like me to go ahead and update the changelog as part of this PR, should I use 2.8.1 as the next version (since this is essentially a bug fix)? I would have submitted this PR against develop, but there didn't seem to be a develop branch.

@StorytellerCZ
Contributor

Yes, v2.8.1. Thanks a lot!

@StorytellerCZ StorytellerCZ added this to the 2.x milestone Nov 23, 2016
@forevermatt
Contributor

@StorytellerCZ Okay, done. Thanks! 😃

@StorytellerCZ StorytellerCZ merged commit 6f33836 into hybridauth:master Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment