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

getUserProfile Permissions as Array #164

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

getUserProfile Permissions as Array #164

wants to merge 4 commits into from

Conversation

cKurultayKalkan
Copy link

Hello All,

I got an error about unsufficient permissions when i dont need user gender, local time or others, so changed permissions as a variable array.

See you!

@pierreavizou
Copy link

pierreavizou commented Jan 6, 2019

I had the same error, it is probably because facebook recently changed their policy, and after January 8, by default, it will be possible to get only the 'id', 'first_name', 'last_name', 'profile_pic' fields. The other fields will require facebook's explicit approval, see the breaking change notice here : https://developers.facebook.com/docs/messenger-platform/identity/user-profile

So I think that the default fields should only be 'id', 'first_name', 'last_name', 'profile_pic', and the user can still add more fields using the array variable. This would allow us to avoid unexpected errors for users that have not requested permissions to view the other fields.

Lock files are important
Add required fields built-in, merge with new permissions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants