search

ueberauth / ueberauth_facebook

Facebook OAuth2 Strategy for Überauth.
MIT License
77 stars 64 forks source link

Updates to use OAuth2 0.8.0 #27

Closed matthewlehner closed 7 years ago

matthewlehner commented 7 years ago

This change was a little bit further reaching than I'd imagined it would be, but it is now working as expected with default options configured.

I haven't tried it with different scopes, so I'm not sure what will be returned.

Please let me know if any changes or improvements need to be made!

matthewlehner commented 7 years ago

Is there anything else needed to get this merged?

doomspork commented 7 years ago

@matthewlehner time in the day to review it 😀

Hoping to look at and merge this today.

matthewlehner commented 7 years ago

Ah – okay, totally not meaning to be a pest! Let me know if there's anything I can do to help! :)

Mazyod commented 7 years ago

This looks great, can hardly wait to see ueberauth transition to OAuth 0.8. Thanks!