-
Notifications
You must be signed in to change notification settings - Fork 63
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
Kuviz publish #718
Kuviz publish #718
Conversation
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.
Some doubts and suggestions.
Co-Authored-By: Alberto Romeu <alrocar@users.noreply.github.com>
Co-Authored-By: Alberto Romeu <alrocar@users.noreply.github.com>
Co-Authored-By: Alberto Romeu <alrocar@users.noreply.github.com>
Co-Authored-By: Alberto Romeu <alrocar@users.noreply.github.com>
3aa42ed
to
64b26ba
Compare
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.
Exciting! Some minor changes. Took me a while to see the bigger picture but the notebook helped a lot :)
if not layer.source.dataset.get_is_saved_in_carto(): | ||
layer.source.dataset.upload(table_name=table_name, context=context) | ||
layer.source = Source(table_name, context=context) | ||
warn('Table `{}` created. In order to publish the map, you will need to create a new Regular API ' |
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.
It'd be pretty sweet if we could help them do that automagically (and explicitly) through cartoframes so that we take away the disruption to the workflow that getting a new key entails.
Related: #542
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.
We don't have Auth API implemented in carto-python yet, but when we have it, we will be able to do it automagically
@@ -188,8 +191,33 @@ def __init__(self, | |||
def _repr_html_(self): | |||
return self._htmlMap.html | |||
|
|||
def publish(self): | |||
pass | |||
def publish(self, name, maps_api_key='default_public', context=None, password=None): |
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.
What's password?
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.
You can creare a custom visualization wirh password. It is the same as a embed with password
Kuviz template
Dataset is immutable
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.
🚀
Solves a part of #674
Publish method implementation.
Please, you should review #717 first because this one is based on it.