passport-facebook icon indicating copy to clipboard operation
passport-facebook copied to clipboard

Published 20 hours ago • verified publisher icon jaredhanson

Updated to v2.8 of graph API

Open fiznool opened this issue 8 years ago • 8 comments
trafficstars

Fixes #180 #198

fiznool avatar Mar 03 '17 11:03 fiznool

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

fiznool avatar Mar 03 '17 11:03 fiznool

Coverage Status

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

coveralls avatar Mar 03 '17 11:03 coveralls

@jaredhanson would it be possible to merge this PR?

paglias avatar Mar 11 '17 11:03 paglias

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

rwky avatar Aug 09 '17 13:08 rwky

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

vctfernandes avatar Sep 16 '17 14:09 vctfernandes

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

ttsirkia avatar Oct 01 '17 19:10 ttsirkia

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 avatar Oct 06 '17 10:10 rwky

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

rwky avatar Jul 07 '18 15:07 rwky