-
Notifications
You must be signed in to change notification settings - Fork 536
Open
Description
This commit in PR #602 has broken the login flow for accounts with a verified email.
To fix one can add "verified" to:
params['fields'] = 'email,first_name,last_name,name,cover,picture'
and make it:
params['fields'] = 'email,first_name,last_name,name,cover,picture,verified'
In django_facebook/connect.py line 70:
email_verified = facebook_data.get('verified', False)
always results in email_verified to be False
I noticed that we are sending version 1 API call here. Any specific reason for that? Just wanted to bring it to your notice since it might get deprecated anytime soon.
Allan-Nava
Metadata
Metadata
Assignees
Labels
No labels