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

No native alert dialog #65

Open
hirbod opened this issue Feb 11, 2015 · 5 comments
Open

No native alert dialog #65

hirbod opened this issue Feb 11, 2015 · 5 comments

Comments

@hirbod
Copy link

hirbod commented Feb 11, 2015

Hi @koraybalci ,

why does this plugin always open the Facebook App instead of showing a native alert dialog like other apps do? The app also does not register inside the Facebook Settings.

It should do something like this:
118d6882-a725-11e4-9403-2bd9380200a0

This is much more intuitive then jumping around a few apps. Can you optimize this behaviour?
Seems like this is just a few lines of code:
Wizcorp/phonegap-facebook-plugin#910

@koraybalci
Copy link
Member

This was the default behaviour back in old Facebook SDK versions. I suppose Facebook decided to just change it and make it parametric.

I found this thread relevant:
http://stackoverflow.com/questions/23476499/native-facebook-login-stopped-working-after-sdk-update-to-3-14

I can add this, but currently I am swamped with other stuff. I'd appreciate a pull request if you can :)

@hirbod
Copy link
Author

hirbod commented Feb 11, 2015

I don't know if I have enough know how for this.

@koraybalci
Copy link
Member

Our login call is at line 290 of CordovaFacebook.m

I would try the parameter mentioned in the link you shared above, however we are not using the same functions. So I should dive in and do some reading and digging and testing.. (not today though)

@hirbod
Copy link
Author

hirbod commented Feb 22, 2015

Any news on this? :)

@hirbod
Copy link
Author

hirbod commented Apr 13, 2015

ping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants