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.

Sign up for GitHub

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

Jump to bottom

Remove 'Alamofire.Manager' subclass #2

Closed

AnthonyMDev opened this issue Apr 4, 2015 · 1 comment

Assignees

Milestone

Owner

AnthonyMDev commented Apr 4, 2015

I'm not sure if this needs to be, or should be, a subclass of 'Alamofire.Manager'.
There are a few other options here.

Create a separate serializer for S3 requests with no dependency on 'Alamofire' and just allow that to be used.
Have a 'requestManager' property on the instance that is used to make all requests. If it is not set, the default manager is used.

I think there may be better options. I'm going to continue considering this.

AnthonyMDev self-assigned this

AnthonyMDev added this to the 0.2.0 milestone

Owner Author

AnthonyMDev commented Apr 6, 2015

This has been completed.

AnthonyMDev closed this as completed

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