Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

Fixed bug in sample_app.py. The access_token needs to be unpacked from returned arguments. #31

Merged
merged 1 commit into from
Dec 17, 2013

Conversation

tonyxiao
Copy link
Contributor

@tonyxiao tonyxiao commented Jul 8, 2012

No description provided.

@ghost
Copy link

ghost commented Dec 18, 2012

Why hasn't tonyxiao's fix been merged into master? The sample app is DOA in its current state.

@rspeed
Copy link

rspeed commented Dec 18, 2012

There doesn't seem to be anyone maintaining this project. I reported it through the main support channels months ago, but got no response.

@petermanser
Copy link

+1

@sursh
Copy link

sursh commented Jun 29, 2013

+1 please merge. Fixes issue #38

@xetch
Copy link

xetch commented Sep 20, 2013

+1

@ekaputra07
Copy link

+1 Please fix it, its totally mislead us.

@johnboiles
Copy link
Contributor

+1, I just wasted 30 minutes on this.

@tonyxiao
Copy link
Contributor Author

Yea I think there's no one monitoring this API project anymore unfortunately.

@rspeed
Copy link

rspeed commented Oct 19, 2013

I told @mikeyk about this a couple weeks ago via Twitter and he promised to have someone take a look.

@rspeed will have our team take a look and merge. Thanks for the heads up

— Mike Krieger (@mikeyk) September 21, 2013
<script async src="//platform.twitter.com/widgets.js" charset="utf-8"></script>

albertotretti added a commit that referenced this pull request Dec 17, 2013
Fixed bug in sample_app.py. The access_token needs to be unpacked from returned arguments.
@albertotretti albertotretti merged commit 5bfd9d6 into facebookarchive:master Dec 17, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants