meteor-phonegap-oauth icon indicating copy to clipboard operation
meteor-phonegap-oauth copied to clipboard

Facebook Tweak

Open HynesIP opened this issue 11 years ago • 8 comments

Anyone had any luck tweaking this plugin for FB? Works great for Twitter.

HynesIP avatar Jun 27 '14 05:06 HynesIP

@HynesIP , hoping to add Facebook, Github, and Google this weekend (Google has partial support currently).

AdamBrodzinski avatar Jun 28 '14 01:06 AdamBrodzinski

Excellent, Adam. Thank you, for the plugin.

HynesIP avatar Jun 28 '14 05:06 HynesIP

I have a partial working branch for facebook here: https://github.com/AdamBrodzinski/meteor-phonegap-oauth/tree/facebook

There's an issue with the nth login not working. The only way to get it to log in again is to uninstall the app and reinstall. Not sure if this is a FB issue or a PhoneGap issue... SO had several references to facebook showing a white screen when trying to login so I suppose it could be related.

The easiest way to try it out would prob be editing the smart.lock file and adding the facebook branch.

AdamBrodzinski avatar Jun 29 '14 04:06 AdamBrodzinski

@HynesIP let me know how it works out. I'll try and work on it again next weekend.

AdamBrodzinski avatar Jun 29 '14 04:06 AdamBrodzinski

Sounds FBish. I'll study the logs and see if I can pinpoint a ill using that branch, and report back. Thank you again!

HynesIP avatar Jun 29 '14 15:06 HynesIP

I didn't pinpoint the cause of the nth login not working, but can say it doesn't occur in Android, which works as expected. I recall a similar issue when adding the FB PhoneGap plugin last year. As you said, a reinstall was a reliable solution. I don't recall if I found a cause or solution in that case. I'll keep looking at it. BTW, I forked the current version and wrapped the android check with a check for device. TY

HynesIP avatar Jul 01 '14 05:07 HynesIP

https://github.com/andrewreedy/meteor-accounts-facebook-cordova Made a package to work with facbeook single sign on / native sdk. Works now.. going to work on making using the graph api easier too :)

andrewreedy avatar Jul 07 '14 17:07 andrewreedy

I managed to get FaceBook to work as well, besides Google and Twitter.

What I've done was to move the "close();" call in file "patch_window.js" from line 89 to the if block right after "Accounts.oauth.tryLoginAfterPopupClosed(credentialToken);"

Can someone else report success as well with this change?

Thanks @AdamBrodzinski for this great package!

nunohvidal avatar Aug 08 '14 14:08 nunohvidal