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

Updated to v2.8 of graph API #199

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

Conversation

fiznool
Copy link

@fiznool fiznool commented Mar 3, 2017

Fixes #180 #198

@fiznool
Copy link
Author

fiznool commented Mar 3, 2017

Some whitespace was trimmed by my code editor so this view offers a more readable PR.

@coveralls
Copy link

coveralls commented Mar 3, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 091f616 on flowxo:master into 2b74dd0 on jaredhanson:master.

@paglias
Copy link

paglias commented Mar 11, 2017

@jaredhanson would it be possible to merge this PR?

@rwky
Copy link

rwky commented Aug 9, 2017

The current api version 2.5 is due to expire in October, would be good to get this merged before then.

@vctfernandes
Copy link

@jaredhanson do you think this needs any further work before merging?

@ttsirkia
Copy link

ttsirkia commented Oct 1, 2017

Any progress with this? October 9th is pretty soon.

@rwky
Copy link

rwky commented Oct 6, 2017

For all those watching this I've created a new PR #220 to update to 2.10 and published an npm module passport-facebook-rwky which contains this PR

@rwky rwky mentioned this pull request Oct 6, 2017
@rwky rwky mentioned this pull request Mar 19, 2018
@rwky
Copy link

rwky commented Jul 7, 2018

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.

7 participants