-
Notifications
You must be signed in to change notification settings - Fork 633
When using enterprise version the "save and edit plot in the cloud" should default to the enterprise domain #444
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
Comments
Hi @lucacerone, great points! For (1), you can now do something like: config(plot_ly(), modeBarButtonsToRemove = list('sendDataToCloud')) For (2), as long as you have the p <- plot_ly() %>% plotly_build()
p$base_url
#> [1] "https://plot.ly" And, if you wanted, you could override this property for a specific plot, and print: p$base_url <- "https://plotly.your-company.com"
p |
Closing for now, but feel free to reopen if you have further issues. |
Hi Carson, However so far I only tried to use ggplotly to convert some ggplot2 plots I I'll try to do some plots from scratch in the future and let you know! Thanks for the help!
|
Try installing the latest version ( |
Thanks, I'll do it first thing tomorrow!
|
Yep, it is a problem specific to ggplotly.
p$base_url is correctly set to my company url. However if I do:
then p$base_url is NULL rather than being my company one (I think it doesn't exist at all the element base_url in p:
returns FALSE. If now I do
it fails with error:
Maybe it depends on how I set the API domain? The company plotly domain is of the form:
So I set "plotly_api_domain" to be: Also, if I try to manually set p$base_url:
shows:
|
Ah, thanks for the detailed report, try reinstalling ( |
Hi, sorry for the late reply. I have just finished installing the latest version v2.4.1 but unfortunately plotly_POST still does not work. In my .Rprofile file I have added:
I then started R (all the environment variables are set and accessible using Sys.getenv("plotly_username") etc etc) I try to follow the very simple example at https://plot.ly/r/ (Run Locally or Publish on the Web):
(by the way the examples on the web are not update to the new and I get the following message:
|
No worries, @lucacerone, my apologizes for the headaches, and thanks for helping out! I think I have a fix in #457, but before I merge it into master, it'd be great if you could confirm that it works on your end. You can install with |
Hi @cpsievert I finally had the time to try the branch you said and it works (meaning I can upload plots to the enterprise version of plotly) However I have noticed two things:
Anyway thanks for the fix, can you let me know when it is merged in the main branch so that I can instll it? Cheers, |
In my company we are running a trial using Plotly Enterprise version.
I have created some HTML reports using Rmarkdown and plotly. They look very nice but they show a button "save and edit plot in the cloud" to share data through plot.ly website. I think in an enterprise environment the default behaviour should be to share plots through the company version of plotly, not on the public one (to avoid potential sharing of confidential data).
Other things I was looking for related to this which I couldn't find:
They are probably not bugs, but I think they would make great enhancements for the enterprise version.
The text was updated successfully, but these errors were encountered: