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

If Facebook returns [], gem should return []. Fixes #350 #351

Closed
wants to merge 1 commit into from
Closed

If Facebook returns [], gem should return []. Fixes #350 #351

wants to merge 1 commit into from

Conversation

kenips
Copy link

@kenips kenips commented Mar 18, 2014

See #350 - Facebook returns "[]" for some responses. Gem should accept them happily in this case?

@kenips kenips changed the title If Facebook returns [], gem should return []. #350 If Facebook returns [], gem should return []. Fixes #350 Mar 18, 2014
@kenips
Copy link
Author

kenips commented Mar 18, 2014

Alternatively we should "patch" Facebook's response and return { data: [] }. Thoughts?

@nov
Copy link
Owner

nov commented Mar 19, 2014

FbGraph::Collection handles [] as {data: []}, so [] should be fine.
BTW, do you have any sample json & sample API call code?

@nov nov closed this in 042cd45 Mar 20, 2014
@kenips kenips deleted the bug/array_response branch March 24, 2014 16:32
@kenips
Copy link
Author

kenips commented Mar 24, 2014

👍 thanks!! Sorry wasn't able to respond earlier. I agree with you that this is a Facebook bug and we actually opened a bug report with them too. However they insisted [] being valid json in this case.

@kenips
Copy link
Author

kenips commented Apr 4, 2014

Hey @nov can we push up v2.7.12 to rubygems.org? Thanks!!

@nov
Copy link
Owner

nov commented Apr 5, 2014

ah, sorry.
I thought I've already released it, but actually not yet :p
I just released it.

@kenips
Copy link
Author

kenips commented Apr 5, 2014

No worries. Thanks again!!!

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.

None yet

2 participants