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

Update README.md #8

Closed
wants to merge 1 commit into from
Closed

Update README.md #8

wants to merge 1 commit into from

Conversation

jrodbx
Copy link

@jrodbx jrodbx commented Dec 3, 2012

  • added cast to eliminate warnings as [AFOAuthClient clientWithBaseURL] returns AFHTTPClient
  • registering AFJSONRequestOperation is not enough as canProcessRequest returns NO unless Accept header is set
  • added necessary "account" reference in success block

* added cast to eliminate warnings as [AFOAuthClient clientWithBaseURL] returns AFHTTPClient
* registering AFJSONRequestOperation is not enough as canProcessRequest returns NO unless Accept header is set
* added necessary "account" reference in success block
@mattt
Copy link
Contributor

mattt commented Dec 28, 2012

Thanks for your pull request, @jrodbx. I've merged in #6, which should address all of the concerns listed here.

@mattt mattt closed this Dec 28, 2012
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.

None yet

2 participants