apply chart-studio.plotly.com and other fixups

archmoj · archmoj · commit d81e59829a66 · 2020-03-26T14:59:36.000-04:00
diff --git a/src/plot_api/plot_config.js b/src/plot_api/plot_config.js
@@ -34,10 +34,10 @@ var configAttributes = {
         valType: 'string',
         dflt: '',
         description: [
-            'When set it determines base URL e.g. \'https://plotly.com\' for',
-            'the \'Edit in Chart Studio\' (aka sendDataToCloud) mode bar button',
+            'When set it determines base URL for',
+            'the \'Edit in Chart Studio\' `showEditInChartStudio`/`showSendToCloud` mode bar button',
             'and the showLink/sendData on-graph link.',
-            'To enable sending your data to Plotly\'s public cloud, you need to',
+            'To enable sending your data to Chart Studio Cloud, you need to',
             'set both `plotlyServerURL` to \'https://chart-studio.plotly.com\' and',
             'also set `showSendToCloud` to true.'
         ].join(' ')
@@ -265,7 +265,7 @@ var configAttributes = {
         dflt: false,
         description: [
             'Should we include a ModeBar button, labeled "Edit in Chart Studio",',
-            'that sends this chart to plotly.com (formerly plot.ly) or another plotly server',
+            'that sends this chart to chart-studio.plotly.com (formerly plot.ly) or another plotly server',
             'as specified by `plotlyServerURL` for editing, export, etc? Prior to version 1.43.0',
             'this button was included by default, now it is opt-in using this flag.',
             'Note that this button can (depending on `plotlyServerURL` being set) send your data',
diff --git a/test/jasmine/tests/config_test.js b/test/jasmine/tests/config_test.js
@@ -505,7 +505,7 @@ describe('config argument', function() {
             Plotly.plot(gd, [], {})
             .then(function() {
                 expect(gd._context.plotlyServerURL).not.toBe('https://plot.ly');
-                expect(gd._context.plotlyServerURL).not.toBe('https://plotly.com');
+                expect(gd._context.plotlyServerURL).not.toBe('https://chart-studio.plotly.com');
                 expect(gd._context.plotlyServerURL).toBe('');
 
                 Plotly.Plots.sendDataToCloud(gd);
@@ -515,15 +515,15 @@ describe('config argument', function() {
             .then(done);
         });
 
-        it('should be able to connect to plotly cloud when set to https://plotly.com', function(done) {
+        it('should be able to connect to Chart Studio Cloud when set to https://chart-studio.plotly.com', function(done) {
             Plotly.plot(gd, [], {}, {
-                plotlyServerURL: 'https://plotly.com'
+                plotlyServerURL: 'https://chart-studio.plotly.com'
             })
             .then(function() {
-                expect(gd._context.plotlyServerURL).toBe('https://plotly.com');
+                expect(gd._context.plotlyServerURL).toBe('https://chart-studio.plotly.com');
 
                 Plotly.Plots.sendDataToCloud(gd);
-                expect(form.action).toBe('https://plotly.com/external');
+                expect(form.action).toBe('https://chart-studio.plotly.com/external');
                 expect(form.method).toBe('post');
             })
             .catch(failTest)
