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

feat: optionally return all raw email addresses for user #13

Open
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@jaredscheib
Copy link

jaredscheib commented Feb 6, 2019

Adopted in part from jaredhanson#27

When a user has multiple github email addresses, only their first one is being returned. This PR adds the strategy constructor option to return all email addresses in their raw response form, to allow for downstream decisions about how to handle multiple email addresses.

@jaredscheib jaredscheib force-pushed the Paperspace:patch/multiple-emails branch from d6f5a33 to 5bc895e Feb 6, 2019

@jaredscheib jaredscheib force-pushed the Paperspace:patch/multiple-emails branch from 1895461 to 7480c3d Feb 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment