Replies: 1 comment
-
Please dont leak internal package names :) |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Not every API is set up to return
"Content-Type":"application/json"
, some return plain/text. I did a test to see how own ajax wrapper package would react:The
body
on the response reflects the API value correctly, as you can see, but this creates a mismatch from what I would expect fromfetchJson
. I expect JSON, not a string.Is this something that lion should handle, and perhaps throw if
response.body
cannot be parsed as JSON, or should this be picked up in our own ajax wrapper package?Beta Was this translation helpful? Give feedback.
All reactions