rails 5.0 upgrade: make content-type header application/json

zooniverse / talk-api

Apache License 2.0

6 stars 0 forks source link

rails 5.0 upgrade: make content-type header application/json #325

Closed Tooyosi closed 4 months ago

Tooyosi commented 4 months ago

Issue was caused as the ’sticky’ boolean was converted to a string and therefore fails the schema validation
Making the request header CONTENT_TYPE be application/json fixes this

yuenmichelle1 commented 4 months ago

nicee :) good find.

yuenmichelle1 commented 4 months ago

One thing to note when we go to Rails 5.0 (when the canary is up, is to make sure any FE/client that hits talk also sends over application/json as a header). I believe it already does, but doesnt hurt to be wary of it. I have kept note on this in my own doc of things to test once canary is up. (Will share later when we get close to canary deployment)