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

Added support for age range. #218

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mhverbakel
Copy link

Fixes #217.

@coveralls
Copy link

coveralls commented Sep 4, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling b9da85f on mhverbakel:master into 90a642b on jaredhanson:master.

rwky added a commit to passport-next/passport-facebook that referenced this pull request Jul 7, 2018
@rwky
Copy link

rwky commented Jul 7, 2018

@jaredhanson
Copy link
Owner

Passport tries to keep profiles normalized to the Portable Contacts schema, or extensions to it such as OpenSocial. The simplifies integration of multiple social providers, by avoiding things that may be specific to an IdP like Facebook.

IdP-specific attributes are always available via profile._json. I'd consider merging this if there were a clear convention around age range support across IdPs.

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

Successfully merging this pull request may close these issues.

5 participants