Add body transform prop to handle body types other than json

[vesparny]

Hi @tkh44. May I help on this one?

[tkh44]

Yes!

[vesparny]

What do you exactly have in mind?

[tkh44]

@vesparny I think I want to add a responseTransformer prop or something like that. The default would be:

(res) => res.json()

The only tricky part is I think we also need to pass in onError and onSuccess handlers to this transformer so that we can hook it into the existing success/error handling.

Maybe something like

(res, onSuccess, onError) => res.json().then(onSuccess).catch(onError)

What do you think?

[tkh44]

I also wonder if it should be more generic so users can do more than just transform the request. Maybe the more generic version can live on a different prop.

[vesparny]

At the moment unfetch supports :

• text
• json
• xml
• blob

what about having an API like:

<Holen 
  body={JSON.stringify({ message: 'hello' })}
  method="POST"
  url="api.startup.com/users"
  type="json"
  transformResponse={(data) => {
   data.isDecorated = true
   return data
 }}
>
  {({data}) => <pre>{data.isDecorated}</pre>}
</Holen>

[tkh44]

That looks great!

[vesparny]

Will work on it tomorrow :)

cheers

[tkh44]

closed in #3