Closed yonaskolb closed 4 years ago
Any comments @jrcmramos?
Sorry for the delay, but everything looking good! ✨ It makes total sense to me as well the responsibility of the APIClient to inject the encoder to the requests 👍 . This way we also remove the knowledge of the Requests about the API Client (which is at a higher level) .
Nice job!
This is a followup to #172 It adds a
jsonEncoder
toAPIClient
that can be customized. It also hides it behind aRequestEncoder
protocol, so other encoders can be used