-
Notifications
You must be signed in to change notification settings - Fork 2
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
Great Demo, Thank you #2
Comments
Yep, it was done a bit on a rush, there's a lot of work in refactoring code, etc, but I'm glad that you find it useful! If you want to make some changes I'm totally open for it :D |
One more important thought as I am now integrating it in a project I am working on. I believe all these classes should be Decodable and remove Alamofire and EVReflection, the implementation will be cleaner and simpler, I understand there is an effort to do this but it will be better hopefully you will actually consider when you have time. Thanks |
I was thinking about it the other day. Now I have a bit more time, I will start working on it. Thanks for the suggestion! |
Excellent and thank you for the prompt response and considering it, I'll be watching the branch :) |
I started working right now adapting the API to Codable/Decodable, in fact it should be working more or less fine, I'm just adapting the Sample code and trying to create Unit tests. You can take a look in |
I've learn't a lot from this, thank you it came to me at the right time, there is still room for improvements. If there was a better way to make callWithToken a bit more generic and avoid the repeated lines of code for each entity that would be awesome but seriously thank you.
The text was updated successfully, but these errors were encountered: