[dashboard] add link to export CSV from dashboard (#3280)

* [dashboard] add link to export CSV from dashboard fixes #1304 * Fix tests
apache · Aug 15, 2017 · 9c6248f · 9c6248f
1 parent 6841697
commit 9c6248f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/superset/assets/javascripts/dashboard/components/SliceCell.jsx b/superset/assets/javascripts/dashboard/components/SliceCell.jsx
@@ -2,6 +2,8 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 
+import { getExploreUrl } from '../../explore/exploreUtils';
+
 const propTypes = {
  slice: PropTypes.object.isRequired,
  removeSlice: PropTypes.func.isRequired,
@@ -40,7 +42,10 @@ function SliceCell({ expandedSlices, removeSlice, slice }) {
  >
  <i className="fa fa-pencil" />
  </a>
- <a href={slice.slice_url} title="Explore chart" data-toggle="tooltip">
+ <a href={getExploreUrl(slice.form_data, 'csv')} title="Export CSV" data-toggle="tooltip">
+ <i className="fa fa-table" />
+ </a>
+ <a href={getExploreUrl(slice.form_data)} title="Explore chart" data-toggle="tooltip">
  <i className="fa fa-share" />
  </a>
  <a

diff --git a/superset/assets/spec/javascripts/dashboard/SliceCell_spec.jsx b/superset/assets/spec/javascripts/dashboard/SliceCell_spec.jsx
@@ -17,8 +17,8 @@ describe('SliceCell', () => {
  React.isValidElement(<SliceCell {...mockedProps} />),
  ).to.equal(true);
  });
- it('renders five links', () => {
+ it('renders six links', () => {
  const wrapper = mount(<SliceCell {...mockedProps} />);
- expect(wrapper.find('a')).to.have.length(5);
+ expect(wrapper.find('a')).to.have.length(6);
  });
 });