-
Notifications
You must be signed in to change notification settings - Fork 508
Integrate native alert permission on iOS instead of jumping around apps #18
Comments
But actually, your login-code is different, as you, @jeduan, have integrated the new Facebook SDK. |
Actually, that code just checks if there's an active session, and doesn't present the login which is sort of different from the native implementation. Apparently the native class is ACAccount and Facebook did use that in their SDK but switched. We'd have to see if there's a way for the SDK to use the Token if the user signs in with ACAccount, otherwise it would only be useful if the app will not use anything else (graph, dialogs, etc) |
I understand. But what about FBSessionLoginBehaviorUseSystemAccountIfPresent This worked with the Facebook SDK < 4. It checked for SystemAccount and fall back to app-switching. |
To get native auth working, in the login function add login.loginBehavior = FBSDKLoginBehaviorSystemAccount; The resulting code block should look like this: FBSDKLoginManager *login = [[FBSDKLoginManager alloc] init]; |
Ok, so I looked into this, and bumped into https://developers.facebook.com/docs/ios/errors#renew says
So basically this is a breaking change as the plugin user would need to handle stuff that is currently handled by the SDK. This is not great, and I'd put it behind an option but we only accept an array of permissions in |
@jeduan I don't know if this is possible and how much extra-work this would cost, but it would be very nice, as of iOS 9, the App ask if it has permission to open Facebook, and after login it ask's the Facebook-App, if it has permission to return back. Something bad on this is, that after loggin in the normal way on iOS 9, the user will see a "return to facebook" message instead of the network carrier name. Really ugly, as it may interupt status-bar-taps (e.g for scrolling up). See my screenshot on my iPhone 5 with iOS 9 Public Beta 3 |
I had FBSDKLoginBehaviorSystemAccount released in our app and after seeing the auth failures come in, I would now agree to using the default auth flow. Unless this plugin can recover and run the default auth method after an FBSDKLoginBehaviorSystemAccount failure it's not worth having as default. I've seen a lot of 'invalid accounts' 'password needs updating' messages where the user's Facebook system account isn't properly setup or needs updating resulting in auth failures. |
I totaly agree. I would just set this up as an additional logon flow for those who need it. Would love to know how e.g tinder with more than 500 mio user handle this, as they use the system logon flow only |
I know there's a way for apps to be linked to a domain and share cookies/login info, so I know for sure this will be fixed. Just give it a couple of weeks. |
Thanks @jeduan! |
Yeah thanks a lot :) |
I meant the fix will come, not from me, but from Facebook :p |
Please provide the native facebook permission alert. "Jumping" around the apps is so ugly and so 2013.
ccsoft/cordova-facebook#65
I would love to do this by myself, but I really don't have enough know how on this. I hope we could see some of this features. If you want, I could open an own ticket for my wishes but I thought this thread is a good place for it.
There was a ticket on the original Facebook plugin.
Wizcorp#910
Seems like this is not a big deal to implement. Only a few code changes inside of the FacebookConnect.m file. I don't know if this has any sideeffects.
The text was updated successfully, but these errors were encountered: