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

- Method added to prototype that handle requested params that does no… #184

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

Conversation

jungleBadger
Copy link

@jungleBadger jungleBadger commented Oct 24, 2016

Method added to prototype that handle requested params that does not exist on FB profile response;

  • Object "processed" added to profile returned object which include all requested params with a null value when FB does not provide it.
  • Test cases
  • IIFE pattern and use strict added to strategy.js
  • Updated GRAPH API reference to v2.8

…t exist on FB profile response;

- Object "processed" added to profile returned object which include all requested params with a null value when FB does not provide it.
- Test cases
- IIFE pattern and use strict added to strategy.js
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 9f6aa61 on jungleBadger:master into 2b74dd0 on jaredhanson:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 9f6aa61 on jungleBadger:master into 2b74dd0 on jaredhanson:master.

@coveralls
Copy link

coveralls commented Oct 24, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 9f6aa61 on jungleBadger:master into 2b74dd0 on jaredhanson:master.

Daniel Abrão added 2 commits November 16, 2016 23:42
- added property nullEmptyResponse to config object to control to null empty response or not
@coveralls
Copy link

coveralls commented Nov 17, 2016

Coverage Status

Coverage decreased (-0.7%) to 99.286% when pulling ca85ba7 on jungleBadger:master into 2b74dd0 on jaredhanson:master.

@jungleBadger
Copy link
Author

sorry about coverage. Tested the change locally on a project and forgot to include a validation before sending it out; Fixed already.

@rwky
Copy link

rwky commented Jul 7, 2018

@jungleBadger if you make a PR against https://github.com/passport-next/passport-facebook is will get looked into.

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.

3 participants