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

Removed response serializer validate call #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ynechaev
Copy link

@ynechaev ynechaev commented Oct 1, 2014

This method was not implemented so caused a compiler error

This method was not implemented so caused a compiler error
@steipete
Copy link
Owner

steipete commented Oct 1, 2014

Didn't someone else just add this PR? I'm just using NSURLSession so not really maintaining this anymore.

@ynechaev
Copy link
Author

ynechaev commented Oct 1, 2014

@steipete what do you mean under PR, can you clarify please?) So, do you recommend to avoid using this component due to lack of support any more?

@steipete
Copy link
Owner

steipete commented Oct 1, 2014

It still works - I just generally recommend to use NSURLSession unless you need iOS 5/6 support.

@ynechaev
Copy link
Author

ynechaev commented Oct 1, 2014

@steipete I use ios 7+ deployment target, so thank you for the advice. I think I will migrate to NSURLSession.

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.

2 participants