New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use Versioncake for API versioning #2209
Conversation
Looks awesome, though I may be a little bias. Let @aantix or I know if you have any comments, questions or issues. |
I'm happy to see this merged, @radar do you have any input? |
@@ -6,6 +6,9 @@ class Engine < Rails::Engine | |||
isolate_namespace Spree | |||
engine_name 'spree_api' | |||
|
|||
config.view_versions = [1] | |||
config.view_version_extraction_strategy = :http_parameter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an invalid value. Valid ones are :query_parameter
, :http_header
and http_accept_parameter
. I assume you mean http_header
?
Fixes #2209 * Unversion zones controller so it can be used with Versioncake for testing purposes * Zones controller using versioncake for versioning * Move other controllers so they make use of Versioncake * Rename all views to make use of Versioncake * No v2 of api yet
Added to master and split_core. |
Fixes #2209 * Unversion zones controller so it can be used with Versioncake for testing purposes * Zones controller using versioncake for versioning * Move other controllers so they make use of Versioncake * Rename all views to make use of Versioncake * No v2 of api yet Conflicts: api/app/controllers/spree/api/v1/base_controller.rb api/app/controllers/spree/api/v1/taxonomies_controller.rb api/app/controllers/spree/api/v1/taxons_controller.rb api/app/views/spree/api/products/index.v1.rabl api/config/routes.rb api/spree_api.gemspec
Really like this approach combining with versioncake, will definitely start recommending people to use versioncake in conjunction with RABL |
Getting rid of the v1 namespace and using versioncake to manage different versions of our API.