Don't check the contents of responseType

[ghost]

The generated code makes a comparison to see if xhr.responseType === 'json' but in my current Chrome, for example, this variable is always set to the empty string.

The generated code should only rely on the content-type header, and even in the absence of this header I would suggest trying to parse it as json anyway, or at least making it an option.

[ghost]

Oh, sorry, I just noticed #13 ...I suggest merging it ;)

[minad]

@chris-kahn Thanks for confirming. Sorry for the issue - I hope this gets fixed soon.

[phadej]

Has @minad or @chris-kahn tested it in some application that the fix actually works?

I'll merge and release #13 now, but I'll insist on adding tests for the next changes.

[minad]

@phadej I tested it, maybe @chris-kahn could recheck too? I agree that automatic testing would be nice. But how do you do automatic browser tests in servant right now?

[phadej]

To clarify: I don't need automatic tests, but something so I can run a command and get a page where I can check things work. See #15

For example servant-swagger-ui has https://github.com/phadej/servant-swagger-ui/blob/master/example/Main.hs

[minad]

Ok, makes sense!

[phadej]

0.9.3 is on Hackage

[ghost]

I did test that it works, yes. Thanks for fixing it quickly!