Merge pull request #329 from UrbanOS-Public/1143-remove-plotly-link

Conditionally removing the plotly help link

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smartcitiesdata/react-discovery-ui",
-  "version": "2.1.34",
+  "version": "2.1.35",
   "description": "React component for dataset discovery UI",
   "main": "./lib/ReactDiscoveryUI.js",
   "repository": {

src/pages/dataset-view/dataset-view.js

@@ -97,7 +97,10 @@ export default class extends Component {
               {this.isNotDatasetDetailsTab() &&
                 <>
                   <a className='helpLink primary-color' target='_blank' href='https://en.wikipedia.org/wiki/SQL_syntax'>SQL Help&nbsp;&nbsp;</a>
-                  <a className='helpLink primary-color' target='_blank' href='https://plotly.com/chart-studio-help/tutorials/#basic'>Plot.ly Help</a>
+                  {this.isVisualizationEnabled() &&
+                    <a id='plotlyhelp' className='helpLink primary-color' target='_blank' href='https://plotly.com/chart-studio-help/tutorials/#basic'>Plot.ly Help</a>
+                  }
+
                 </>}
             </span>
             {this.isNotDatasetDetailsTab() &&

src/pages/dataset-view/dataset-view.test.js

@@ -1,5 +1,5 @@
 import { shallow } from 'enzyme'
-import { Tab, TabPanel } from 'react-tabs'
+import { Tab, TabPanel, TabList } from 'react-tabs'
 
 import DatasetView from './dataset-view'
 import QueryView from '../query-view'
@@ -82,6 +82,11 @@ describe('Visualizations when DISABLE_VISUALIZATIONS is true', () => {
   it('does not have the visualizations view', () => {
     expect(subject.find(ChartView).length).toEqual(0)
   })
+  it('does not have the plot.ly help link', () => {
+    subject.setState({ index: 1 })
+    const test = subject.find('#plotlyhelp')
+    expect(test.length).toEqual(0)
+  })
 })
 
 describe('dataset view when dataset is not loaded', () => {