Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Requesting Pull #9

Merged
merged 1 commit into from Jun 5, 2011

Conversation

Projects
None yet
2 participants
Contributor

tenpaiyomi commented May 6, 2011

philippec,

Love the framework, but noted that it didn't have the ability to make POST requests, which some of the FB requests require (such as publishing to the users wall), so I modified the code slightly to allow such, by simply passing a YES/NO to the sendRequest:params (now sendRequest:params:post) method.

Adding a post option to the sendRequest:params method, such that user…
…s can make requests that require a POST url request
Owner

philippec commented May 25, 2011

Any news? I'd like to integrate your changes :-)

philippec added a commit that referenced this pull request Jun 5, 2011

Merge pull request #9 from tenpaiyomi/master
Added ability to make POST requests

@philippec philippec merged commit be4b329 into philippec:master Jun 5, 2011

Owner

philippec commented Jun 5, 2011

I decided to pull it and perform the minor cleanup we discussed above. That way we don't have an outstanding pull request for too long.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment