search

ccsoft / cordova-facebook

Cordova plugin that handles Facebook integration in mobile apps.
Apache License 2.0
87 stars 43 forks source link

Login behaviour optional with setAskForAllPermissionsAtOnce #42

Open ptbrowne opened 10 years ago

ptbrowne commented 10 years ago

Hi,

I just reinstalled the plugin and it seems that now, Facebook asks for the app to be able to write on the wall. Is it something expected ? Can we deactivate this behaviour ?

Thanks for your plugin, it is very useful !

koraybalci commented 10 years ago

That's right, we are showing a popup to preview before posting to wall now on Android. It was like this all along in iOS by the way. At the moment, it is not optional although we can make it so. However, I don't know if we can make it optional in iOS.

ptbrowne commented 10 years ago

That would be great ! I was ready to try and do and pull request to be able to configure the asking but I could not find where it was... Can you point me to the relevant section ? Thanks

koraybalci commented 10 years ago

Now I check back and realize you might be asking something else.. Are you asking, durinng login, asking for the write permission? If so, this is controlled by setAskForAllPermissionsAtOnce call during init phase.

If you are asking about the preview popup during posting to wall, this is controlled by simple facebook lib we rely on on Android side, I am afraid we cannot tune that behavior at the moment.

ptbrowne commented 10 years ago

Indeed, I was asking about the login, sorry if it was unclear. Do you think it would be OK to add a parameter in the options of the plugin to override the default value of true of setAskForAllPermissionsAtOnce ? I would be glad to do the pull request :)

koraybalci commented 10 years ago

Yes, but we should think about the iOS implementation as well. I am not even sure if it is possible.