Django-facebook icon indicating copy to clipboard operation
Django-facebook copied to clipboard

Login flow broke by a recent commit

Open masterchiefag opened this issue 7 years ago • 1 comments

This commit in PR #587 has broken the login flow.

The problematic change is:

(Earlier)

      self._me = me = self.get('me', fields="id,name,email,verified")

(Now)

      self._me = me = self.get('me')

In django_facebook/connect.py line 69: email = facebook_data.get('email', False) email comes out as False here and as a result the user lands to error page saying email field is required. Verified it by reverting this change.

masterchiefag avatar Mar 16 '17 12:03 masterchiefag

That analysis looks correct to me. I've just pushed a change on master.

About Maintenance & Django-Facebook

Django and Facebook are both rapidly changing at the moment. Meanwhile, I'm caught up in a startup and don't have much spare time. The library needs a good round of testing against the latest python, django and facebook graph API. Contributions are strongly appreciated. Seriously, give github a try, fork and get started :)

tschellenbach avatar Mar 16 '17 15:03 tschellenbach