mirror of
https://git.mirrors.martin98.com/https://github.com/SigNoz/signoz
synced 2025-08-15 22:25:54 +08:00
fix: enable deleting selected columns (#3061)
Co-authored-by: Vishal Sharma <makeavish786@gmail.com>
This commit is contained in:
parent
0750231b4b
commit
d4b95b4848
@ -1,7 +1,6 @@
|
|||||||
import { RadioChangeEvent } from 'antd';
|
import { RadioChangeEvent } from 'antd';
|
||||||
import { getAggregateKeys } from 'api/queryBuilder/getAttributeKeys';
|
import { getAggregateKeys } from 'api/queryBuilder/getAttributeKeys';
|
||||||
import { QueryBuilderKeys } from 'constants/queryBuilder';
|
import { QueryBuilderKeys } from 'constants/queryBuilder';
|
||||||
import { useNotifications } from 'hooks/useNotifications';
|
|
||||||
import useUrlQueryData from 'hooks/useUrlQueryData';
|
import useUrlQueryData from 'hooks/useUrlQueryData';
|
||||||
import { useCallback, useEffect, useMemo } from 'react';
|
import { useCallback, useEffect, useMemo } from 'react';
|
||||||
import { useQuery } from 'react-query';
|
import { useQuery } from 'react-query';
|
||||||
@ -29,8 +28,6 @@ const useOptionsMenu = ({
|
|||||||
aggregateOperator,
|
aggregateOperator,
|
||||||
initialOptions = {},
|
initialOptions = {},
|
||||||
}: UseOptionsMenuProps): UseOptionsMenu => {
|
}: UseOptionsMenuProps): UseOptionsMenu => {
|
||||||
const { notifications } = useNotifications();
|
|
||||||
|
|
||||||
const {
|
const {
|
||||||
query: optionsQuery,
|
query: optionsQuery,
|
||||||
queryData: optionsQueryData,
|
queryData: optionsQueryData,
|
||||||
@ -105,18 +102,12 @@ const useOptionsMenu = ({
|
|||||||
({ id }) => id !== columnKey,
|
({ id }) => id !== columnKey,
|
||||||
);
|
);
|
||||||
|
|
||||||
if (!newSelectedColumns.length) {
|
|
||||||
notifications.error({
|
|
||||||
message: 'There must be at least one selected column',
|
|
||||||
});
|
|
||||||
} else {
|
|
||||||
redirectWithOptionsData({
|
redirectWithOptionsData({
|
||||||
...defaultOptionsQuery,
|
...defaultOptionsQuery,
|
||||||
selectColumns: newSelectedColumns,
|
selectColumns: newSelectedColumns,
|
||||||
});
|
});
|
||||||
}
|
|
||||||
},
|
},
|
||||||
[optionsQueryData, notifications, redirectWithOptionsData],
|
[optionsQueryData, redirectWithOptionsData],
|
||||||
);
|
);
|
||||||
|
|
||||||
const handleFormatChange = useCallback(
|
const handleFormatChange = useCallback(
|
||||||
|
Loading…
x
Reference in New Issue
Block a user