Skip to content

Commit

Permalink
fix errors of resolving conflicts
Browse files Browse the repository at this point in the history
  • Loading branch information
josedev-union committed Dec 4, 2024
1 parent f9059ca commit 7e33231
Show file tree
Hide file tree
Showing 9 changed files with 11 additions and 49 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -230,9 +230,6 @@ export function transformSeries(
if (stackId && stackIdSuffix) {
stackId += stackIdSuffix;
}
if (stackId && stackIdSuffix) {
stackId += stackIdSuffix;
}
let plotType;
if (
!isConfidenceBand &&
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,20 +23,18 @@ import {
D3_TIME_FORMAT_DOCS,
DEFAULT_TIME_FORMAT,
formatSelectOptions,
sections,
sharedControls,
} from '@superset-ui/chart-controls';
import { showValueControl } from '../controls';

const config: ControlPanelConfig = {
controlPanelSections: [
sections.genericTime,
{
label: t('Query'),
expanded: true,
controlSetRows: [
[hasGenericChartAxes ? 'x_axis' : null],
[hasGenericChartAxes ? 'time_grain_sqla' : null],
['x_axis'],
['time_grain_sqla'],
['groupby'],
['metric'],
['adhoc_filters'],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@ function SearchInput({ count, value, onChange }: SearchInputProps) {
className="form-control input-sm"
placeholder={tn('search.num_records', count)}
value={value}
aria-label={t('Search %s records', count)}
onChange={onChange}
/>
</span>
Expand Down
2 changes: 1 addition & 1 deletion superset-frontend/src/components/Slider/Slider.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -36,4 +36,4 @@ test('should render with default value on tooltip', () => {
expect(
screen.getAllByText(`${mockedProps.defaultValue}`)[0],
).toBeInTheDocument();
});
});
7 changes: 7 additions & 0 deletions superset-frontend/src/dashboard/reducers/dashboardState.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ import {
SET_FOCUSED_FILTER_FIELD,
UNSET_FOCUSED_FILTER_FIELD,
SET_ACTIVE_TAB,
SET_ACTIVE_TABS,
SET_FULL_SIZE_CHART_ID,
ON_FILTERS_REFRESH,
ON_FILTERS_REFRESH_SUCCESS,
Expand Down Expand Up @@ -188,6 +189,12 @@ export default function dashboardStateReducer(state = {}, action) {
activeTabs: Array.from(newActiveTabs),
};
},
[SET_ACTIVE_TABS]() {
return {
...state,
activeTabs: action.activeTabs,
};
},
[SET_OVERRIDE_CONFIRM]() {
return {
...state,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
*/

import dashboardStateReducer from './dashboardState';
import { setActiveTab } from '../actions/dashboardState';
import { setActiveTab, setActiveTabs } from '../actions/dashboardState';

describe('DashboardState reducer', () => {
it('SET_ACTIVE_TAB', () => {
Expand Down
6 changes: 0 additions & 6 deletions superset-frontend/src/features/alerts/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,12 +62,6 @@ export type SlackChannel = {
is_private: boolean;
};

export type NotificationSetting = {
method?: NotificationMethodOption;
recipients: string;
options: NotificationMethodOption[];
};

export type Recipient = {
recipient_config_json: {
target: string;
Expand Down
20 changes: 0 additions & 20 deletions superset-frontend/src/pages/RowLevelSecurityList/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -299,26 +299,6 @@ function RowLevelSecurityList(props: RLSProps) {
),
paginate: true,
},
{
Header: t('Modified by'),
key: 'changed_by',
id: 'changed_by',
input: 'select',
operator: FilterOperator.relationOneMany,
unfilteredLabel: t('All'),
fetchSelects: createFetchRelated(
'rowlevelsecurity',
'changed_by',
createErrorHandler(errMsg =>
t(
'An error occurred while fetching dataset datasource values: %s',
errMsg,
),
),
user,
),
paginate: true,
},
],
[user],
);
Expand Down
13 changes: 0 additions & 13 deletions superset-frontend/src/pages/SavedQueryList/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -381,19 +381,6 @@ function SavedQueryList({
accessor: 'changed_on_delta_humanized',
size: 'xl',
},
{
Cell: ({
row: {
original: {
changed_by: changedBy,
changed_on_delta_humanized: changedOn,
},
},
}: any) => <ModifiedInfo user={changedBy} date={changedOn} />,
Header: t('Last modified'),
accessor: 'changed_on_delta_humanized',
size: 'xl',
},
{
Cell: ({ row: { original } }: any) => {
const handlePreview = () => {
Expand Down

0 comments on commit 7e33231

Please sign in to comment.