QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-04-07 10:31:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: disable text reports for now (#16257)

Browse Source 
* fix: disable text reports for now

* Update tests
This commit is contained in:
Beto Dealmeida
2021-08-16 08:16:48 -07:00
committed by GitHub
parent 542b864e61
commit ee9a384758
3 changed files with 2 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
superset-frontend/src/components/ReportModal/index.tsx
View File

@@ -126,12 +126,7 @@ type ReportActionType =
};
const DEFAULT_NOTIFICATION_FORMAT = 'TEXT';
const TEXT_BASED_VISUALIZATION_TYPES = [
'pivot_table',
'pivot_table_v2',
'table',
'paired_ttest',
];
const TEXT_BASED_VISUALIZATION_TYPES: string[] = [];
const reportReducer = (
state: Partial<ReportObject> | null,

16
superset-frontend/src/views/CRUD/alert/AlertReportModal.test.jsx
View File

@@ -260,22 +260,6 @@ describe('AlertReportModal', () => {
expect(wrapper.find(Radio)).toHaveLength(2);
});
it('renders text option for text-based charts', async () => {
const props = {
...mockedProps,
alert: mockData,
};
const textWrapper = await mountAndWait(props);
const chartOption = textWrapper.find('input[value="chart"]');
act(() => {
chartOption.props().onChange({ target: { value: 'chart' } });
});
await waitForComponentToPaint(textWrapper);
expect(textWrapper.find('input[value="TEXT"]')).toExist();
});
it('renders input element for working timeout', () => {
expect(wrapper.find('input[name="working_timeout"]')).toExist();
});

7
superset-frontend/src/views/CRUD/alert/AlertReportModal.tsx
View File

@@ -52,12 +52,7 @@ import {
const SELECT_PAGE_SIZE = 2000; // temporary fix for paginated query
const TIMEOUT_MIN = 1;
const TEXT_BASED_VISUALIZATION_TYPES = [
'pivot_table',
'pivot_table_v2',
'table',
'paired_ttest',
];
const TEXT_BASED_VISUALIZATION_TYPES: string[] = [];
type SelectValue = {
value: string;