search

spring-attic / spring-social-facebook

Facebook API binding and connect support.
http://projects.spring.io/spring-social-facebook
Apache License 2.0
248 stars 269 forks source link

Added a field full_picture to the Post Class. #213

Open gokul1794 opened 8 years ago

gokul1794 commented 8 years ago

Fixes #211
This will help when someone needs the original image rather than the default image provided by facebook which cannot be practically used anywhere else due to its size. I faced this problem as I needed to fetch Posts for my app, where I needed the message and the picture, but the picture returned by the current Post object was of a size that I could not use. Fetching full sized images using fetchConnections was slowing down the time taken to send the data back to my app as a new request needs to be made for every post.

marky-mark commented 8 years ago

+1 works for reads too

habuma commented 7 years ago

@gokul1794 : I'm ready to merge this, but before I do, can I ask for some tests (or updates to existing tests) to verify what this PR claims to do?