-
Notifications
You must be signed in to change notification settings - Fork 655
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
deploying apps #15595
deploying apps #15595
Conversation
25f0c3c
to
a40883a
Compare
My own acceptance:
I am removing |
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.
LGTM! I left a few suggestions and thoughts.
@@ -39,6 +39,7 @@ class Admin::VisualizationsController < Admin::AdminController | |||
|
|||
before_filter :resolve_visualization_and_table_if_not_cached, only: [:embed_map] | |||
before_filter :redirect_to_kuviz_if_needed, only: [:embed_map] | |||
before_filter :redirect_to_app_if_needed, only: [:embed_map] |
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.
I guess there's no need to worry about /embed_map
here, but it's not harmful anyway...
Co-Authored-By: Gonzalo Riestra <gonzalor@cartodb.com>
62c3176
to
bce669d
Compare
Cache acceptance:
|
Ready! It will be deployed tomorrow morning |
https://app.clubhouse.io/cartoteam/story/67811/backend-for-deploying-apps
Summary:
It needs to be deployed with https://github.com/CartoDB/cartodb-platform/pull/6475