Skip to content

Re-add proper boolean support for object subclassing. #21

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

Merged
merged 1 commit into from
Aug 15, 2015

Conversation

richardjrossiii
Copy link
Contributor

Fixes #13. This was broken due to NSJSONSerialization serializing all NSNumbers as '0' and '1', but any CFBooleans as true and false. We now re-add explicit support for booleans.

@nlutsenko
Copy link
Contributor

LGTM, single nit.

@richardjrossiii richardjrossiii force-pushed the richardross.bool.typeencoding branch from d9e0115 to dc18d85 Compare August 15, 2015 00:01
Fixes #13. This was broken due to `NSJSONSerialization` serializing all `NSNumbers` as '0' and '1', but any `CFBoolean`s as `true` and `false`. We now re-add explicit support for booleans.
@richardjrossiii richardjrossiii force-pushed the richardross.bool.typeencoding branch from dc18d85 to cbe4713 Compare August 15, 2015 00:30
richardjrossiii added a commit that referenced this pull request Aug 15, 2015
Re-add proper boolean support for object subclassing.
@richardjrossiii richardjrossiii merged commit 17c5dc7 into master Aug 15, 2015
@richardjrossiii richardjrossiii deleted the richardross.bool.typeencoding branch August 15, 2015 00:43
@emonster
Copy link

Thank you for fixing this issue!!
Where do I get the fix? Can I get it by pod? or where can I download the new Parse.framework?

@richardjrossiii
Copy link
Contributor Author

You can change your pod file to use:

pod 'Parse', :git => "https://github.com/parseplatform/parse-sdk-iOS-OSX.git"

Note that this will make you run off of master for parse, which is not guaranteed to be stable. You could specify a specific commit, details for which can be found on the cocoa pods website.

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.

4 participants