Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update omniauth strategy to pass location #78

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

marcusguttenplan
Copy link

Adding additional location field from raw_info to the object that is created so that it better fits into Omniauth's Auto Hash Schema

@ac21
Copy link

ac21 commented Apr 3, 2019

+1
any chance we could commit this?

@marcusguttenplan
Copy link
Author

Sadly this project isn't being actively maintained and I don't have write access. Feel free to use my fork if that works for you! It's a super minimal change.

@suprnova32
Copy link
Member

@marcusguttenplan thanks for submitting this PR. I just started reviewing old PRs here, and we can merge yours if you add a small spec to check if the location info is added correctly.

@BobbyMcWho
Copy link
Member

@marcusguttenplan are you still interested in getting this merged? We just need a spec added.

@marcusguttenplan
Copy link
Author

I can tackle this soon, sorry for the delay!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants