Updated to v2.8 of graph API

jaredhanson / passport-facebook

Facebook authentication strategy for Passport and Node.js.

https://www.passportjs.org/packages/passport-facebook/?utm_source=github&utm_medium=referral&utm_campaign=passport-facebook&utm_content=about

MIT License

1.29k stars 447 forks source link

Updated to v2.8 of graph API #199

Open fiznool opened 7 years ago

fiznool commented 7 years ago

Fixes #180 #198

fiznool commented 7 years ago

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

coveralls commented 7 years ago

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

paglias commented 7 years ago

@jaredhanson would it be possible to merge this PR?

rwky commented 7 years ago

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

vctfernandes commented 6 years ago

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

ttsirkia commented 6 years ago

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

rwky commented 6 years ago

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 commented 6 years ago

Merged in https://github.com/passport-next/passport-facebook