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

Getting Facebook Picture in 'normal' size that is not the default size #76

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

olapaola
Copy link

@olapaola olapaola commented Mar 5, 2014

Pre-specified size of picture to normal size. The default size will be square (50x50) if you don't pre-specified it and it results small sometimes. Normal size is 100x100

@jaredhanson
Copy link
Owner

Is there any reference to the .type(normal) modifier in Facebook's documentation? I'd like to add a link to this in the source code comments to clarify intent.

@olapaola
Copy link
Author

olapaola commented Mar 9, 2014

You can find it in the Graphic API Explorer Tool: https://developers.facebook.com/tools/explorer. Searching for "picture", modifier: "type" , and value "normal".

@olapaola
Copy link
Author

olapaola commented Dec 4, 2015

Link to Facebook's documentation with a reference to type "normal" reference: https://developers.facebook.com/docs/graph-api/reference/user/picture.

I saw it finally there and I remembered my pull request ;).

rwky added a commit to passport-next/passport-facebook that referenced this pull request Jul 7, 2018
@rwky
Copy link

rwky commented Jul 7, 2018

@rwky
Copy link

rwky commented Jul 7, 2018

Scratch that this has failing tests @olapaola if you make a PR against https://github.com/passport-next/passport-facebook is will get looked into.

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.

3 participants