search

mediapop / django-friendface

Getting Facebook to work for Django.
2 stars 0 forks source link

Like gate mixin #46

Closed kitsunde closed 11 years ago

kitsunde commented 11 years ago

One potential issue here @gaqzi is if you are relying on the fql method to return a list. The newer FQL endpoint returns {data: []}.

gaqzi commented 11 years ago

As far as I remember I've never used any FQL method manually. The question I have though is if there'd be a point in returning {data: []} over just the list? Or is it there to allow for pagination?

kitsunde commented 11 years ago

I don't think FQL does pagination at all, but if FQL returns {data :[]} we should return the same. Otherwise if they choose to decorate their root object, we would have to either change all of our code to accomodate a new format, or boilerplate on a true_fql.

gaqzi commented 11 years ago

on fql sounds good!

kitsunde commented 11 years ago

Updated ping @gaqzi