Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Remove unnecessary json encoding#563

Open
crazytonyli wants to merge 4 commits intotrunkfrom
remove-unnecessary-json-encoding
Open

Remove unnecessary json encoding#563
crazytonyli wants to merge 4 commits intotrunkfrom
remove-unnecessary-json-encoding

Conversation

@crazytonyli
Copy link
Contributor

Description

I noticed quite a few places that parse the HTTP response as a JSON object and then turn the JSON object into a JSON string so that it can be used with Swift's Decoder. This PR removes the unnecessary round trip and parse the HTTP response directly using Swift's Decoder.


  • Please check here if your pull request includes additional test coverage.
  • I have considered updating the version in the .podspec file.
  • I have considered if this change warrants release notes and have added them to the appropriate section in the CHANGELOG.md if necessary.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant