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

[WIP] 2.0.0 #39

Merged
merged 38 commits into from
Jun 9, 2016
Merged

[WIP] 2.0.0 #39

merged 38 commits into from
Jun 9, 2016

Conversation

isair
Copy link
Owner

@isair isair commented Jun 14, 2015

A better structured JSONHelper with support for serialization (will be completed with 2.1.0) as well as deserialization. Also adds additional type conversions that were not supported previously.

Closes #37, closes #23, closes #14, closes #7, closes #43.

Review on Reviewable

┆Issue is synchronized with this Asana task

Fixed int to bool conversion.
@isair isair removed the in progress label Nov 23, 2015
@isair
Copy link
Owner Author

isair commented Nov 23, 2015

Reviewed 3 of 9 files at r1, 15 of 18 files at r2.
Review status: all files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.


JSONHelper/Extensions/Bool.swift, line 43 [r2] (raw file):
Add "yes" and "y".


JSONHelper/Extensions/Bool.swift, line 45 [r2] (raw file):
Add "no" and "n".


JSONHelper/Extensions/Color.swift, line 11 [r2] (raw file):
Add support for alpha then support inputs of length 3, 4, and 8.


JSONHelper/Extensions/String.swift, line 35 [r2] (raw file):
Convert from Double and NSNumber.


Comments from the review on Reviewable.io

@isair isair force-pushed the dev-2.0.0 branch 6 times, most recently from 4d5d759 to a153bfc Compare January 23, 2016 05:31
Additionally:
- Bumped version to 2.0.0.
- Completed more tests.
- Minor code refactoring.
- Commented out JSONAPI code temporarily.
These commits will be squashed, so I'm going to detail these later on.
Additional changes:
- NSDate to String conversion.
- Serializable protocol.
@isair isair force-pushed the dev-2.0.0 branch 5 times, most recently from c00e77d to 9967b14 Compare April 18, 2016 02:53
Except serialization and JSON API support.
# Conflicts:
#	JSONHelper/JSONHelper.swift
@isair isair merged commit 63e0a69 into master Jun 9, 2016
@isair isair deleted the dev-2.0.0 branch June 17, 2016 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants