Deserialize nil to `[NSNull null]`

SkygearIO / skygear-SDK-iOS

Skygear SDK for iOS

https://skygear.io

Other

10 stars 28 forks source link

Deserialize nil to `[NSNull null]` #73

Closed ben181231 closed 7 years ago

ben181231 commented 7 years ago

connect #71

oursky-travis commented 7 years ago

@ben181231, thanks for your PR! By analyzing the history of the files in this pull request, we identified @cheungpat and @rickmak to be potential reviewers.

cheungpat commented 7 years ago

I added a test case to demonstrate a case when date format is bad.

This will cause similar error described in #64. I think setting the field to NSNull is good. We should also print the problem to console log.

ben181231 commented 7 years ago

@cheungpat The PR is updated.