search

richhollis / swagger-docs

Generates swagger-ui json files for Rails APIs with a simple DSL.
MIT License
750 stars 150 forks source link

Change success status to ok #89

Closed ck3g closed 9 years ago

ck3g commented 10 years ago

To receive 200 response code Rack::Utils.status_code should receive :ok as argument value instead of :success.

http://rubydoc.info/github/rack/rack/Rack/Utils

ck3g commented 10 years ago

Now I'm not sure if deprecation is a good thing =) We can get rid of that warning and just leave it as an alias. Or keep :success in docs but change it to :ok under the hood before passing to the Rack::Utils.status_code

richhollis commented 10 years ago

Good spot - I will update this shortly.

richhollis commented 9 years ago

Hey @ck3g - thanks for this. I've removed the deprecation and just replaced it if :success is used.