-
Notifications
You must be signed in to change notification settings - Fork 388
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
Data instead of JSON - anyone wanna work on this with me? #345
Comments
as a result, \t is appearing as tab control character, breaking JSONDecoder decoding |
@meteochu @mattdonnelly hi guys! i'm super grateful for your work, if you have a minute, I would appreciate some guidance: Why is decoding done with a bespoke JSON type, when using JSONDecoder with Swift Data type is so easy and convenient? |
I need this as well and replaced the json with data in my fork. |
@steipete any chance you can share your fork? |
This is like a 10 minute edit. My fork contains far more changes done specifically for my purpose, makes no sense to open source. |
I submitted a PR for this back in January. Feel free to take a look at it and see if it works for you: #341 |
Is this because Decodable wasn't available before?
I'd like swifter to produce a Data instance which I can use JSONDecoder with. Anyone wanna collaborate on this?
not sure how to decode this JSON so I'm having to stringify with json.description first which seems like a terrible idea
The text was updated successfully, but these errors were encountered: