-
Notifications
You must be signed in to change notification settings - Fork 1
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
Facebook data store #37
Conversation
6e36420
to
567d596
Compare
testapp/models.py
Outdated
|
||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit-pick but can we remove this extra whitespace?
testapp/views.py
Outdated
|
||
#graph = facebook.GraphAPI(access_token=myToken, version=2.10) | ||
#profile = graph.get_object('me') | ||
#args = {'fields' : 'address'} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this bit that commented out? If not can we remove it?
testapp/views.py
Outdated
if k == 'age_range': | ||
user_info.age_range = fb_user_info['age_range']['min'] | ||
if k == 'religion': | ||
user_info.religion = fb_user_info['religion'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there might be a better approach to this, like traversing the JSON in some other way but we can come back to this for the sake of time
testapp/views.py
Outdated
|
||
#def get_facebook_account(): | ||
#user = SocialAccount.objects.get(uid="10210039294274709") | ||
#access_token = SocialToken.objects.filter(account__user=user.user, account__provider='facebook').first() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we remove this? Or are we still saving data this way?
Also looks like there's some merge conflicts here; once you address the other comments I made I can go through the conflict stuff |
567d596
to
e19fe93
Compare
did the formatting stuff and ill look into a cleaner way to parse the json |
Changes to models, views, and admin to save user info from facebook graph api Added education to User_Info field Formatting fix: unnecessary spaces
e19fe93
to
1b3885e
Compare
No description provided.