mirror of
https://git.mirrors.martin98.com/https://github.com/SigNoz/signoz
synced 2025-07-24 05:14:30 +08:00
fix: empty string search doesn't return all exceptions (#4346)
* fix: empty string search doesn't return all exceptions * fix: add type
This commit is contained in:
parent
6f5f361a7e
commit
1e679a0d64
@ -48,6 +48,15 @@ import {
|
||||
urlKey,
|
||||
} from './utils';
|
||||
|
||||
type QueryParams = {
|
||||
order: string;
|
||||
offset: number;
|
||||
orderParam: string;
|
||||
pageSize: number;
|
||||
exceptionType?: string;
|
||||
serviceName?: string;
|
||||
};
|
||||
|
||||
function AllErrors(): JSX.Element {
|
||||
const { maxTime, minTime, loading } = useSelector<AppState, GlobalReducer>(
|
||||
(state) => state.globalTime,
|
||||
@ -162,16 +171,23 @@ function AllErrors(): JSX.Element {
|
||||
filterKey,
|
||||
filterValue || '',
|
||||
);
|
||||
history.replace(
|
||||
`${pathname}?${createQueryParams({
|
||||
order: updatedOrder,
|
||||
offset: getUpdatedOffset,
|
||||
orderParam: getUpdatedParams,
|
||||
pageSize: getUpdatedPageSize,
|
||||
exceptionType: exceptionFilterValue,
|
||||
serviceName: serviceFilterValue,
|
||||
})}`,
|
||||
);
|
||||
|
||||
const queryParams: QueryParams = {
|
||||
order: updatedOrder,
|
||||
offset: getUpdatedOffset,
|
||||
orderParam: getUpdatedParams,
|
||||
pageSize: getUpdatedPageSize,
|
||||
};
|
||||
|
||||
if (exceptionFilterValue && exceptionFilterValue !== 'undefined') {
|
||||
queryParams.exceptionType = exceptionFilterValue;
|
||||
}
|
||||
|
||||
if (serviceFilterValue && serviceFilterValue !== 'undefined') {
|
||||
queryParams.serviceName = serviceFilterValue;
|
||||
}
|
||||
|
||||
history.replace(`${pathname}?${createQueryParams(queryParams)}`);
|
||||
confirm();
|
||||
},
|
||||
[
|
||||
@ -198,8 +214,10 @@ function AllErrors(): JSX.Element {
|
||||
<Input
|
||||
placeholder={placeholder}
|
||||
value={selectedKeys[0]}
|
||||
onChange={(e): void =>
|
||||
setSelectedKeys(e.target.value ? [e.target.value] : [])
|
||||
onChange={
|
||||
(e): void => setSelectedKeys(e.target.value ? [e.target.value] : [])
|
||||
|
||||
// Need to fix this logic, when the value in empty, it's setting undefined string as value
|
||||
}
|
||||
allowClear
|
||||
defaultValue={getDefaultFilterValue(
|
||||
|
Loading…
x
Reference in New Issue
Block a user