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

Fix converting classes to Json that are part of "sub bundles" #34

Closed
wants to merge 1 commit into from

Conversation

mikumi
Copy link

@mikumi mikumi commented Jul 14, 2015

Fixes #23

Fix converting classes to Json that are part of 'sub bundles' of the main app (e.g. unit tests or cocoa pods).

This will:

Feedback appreciated. Please let me know if there is a better solution :-)

…art of the main app (e.g. unit tests or cocoa pods)
@aryaxt
Copy link
Owner

aryaxt commented Jul 14, 2015

Awesome thanks, will look into it

@mikumi mikumi closed this Jun 7, 2016
@aryaxt
Copy link
Owner

aryaxt commented Jun 7, 2016

sorry completely forgot to get back to this, and haven't looked into my repos forever?
was there a problem with the code? have you been using it?

@bestimmaa
Copy link

Same problem here, will give this a try!

@mikumi
Copy link
Author

mikumi commented Jun 30, 2016

Sorry, I can't provide much input anymore. When I closed this PR 3 weeks ago, I couldn't even remember I ever used OCMapper, or what it does (no offense! :-). @bestimmaa I hope the proposed fix still helps though, maybe you could leave a quick note?

@bestimmaa
Copy link

Of course I will report back :) Thanks for all the work you put into this. Anything else you are using for object mapping now? Like your lightweight approach but still exploring possibilities

@mikumi
Copy link
Author

mikumi commented Jul 1, 2016

Hm I have used https://github.com/Mantle/Mantle a few times, but it looks like it's designed for Objective-C. But to be honest I have often also just rolled my own manual json<->object mapping solutions

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.

3 participants