0ct0pu5/kafka-ui
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed Selected partitions label (#2042)

Browse source 
* Fixed Selected partitions label

* Console logs removed
This commit is contained in:
Kirill Morozov 2022-05-30 13:24:59 +03:00 • committed by GitHub
parent 4f1078aabb
commit 996e127a02
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/Filters.tsx
View file

@ -181,7 +181,7 @@ const Filters: React.FC<FiltersProps> = ({
partitions.map((partition: Partition) => { partitions.map((partition: Partition) => {
return { return {
value: partition.partition, value: partition.partition,
label: String(partition.partition), label: `Partition #${partition.partition.toString()}`,
}; };
}) })
); );

4
kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/utils.ts
View file

@ -53,7 +53,7 @@ export const getSelectedPartitionsFromSeekToParam = (
if (selectedPartitionIds?.includes(partition)) { if (selectedPartitionIds?.includes(partition)) {
return { return {
value: partition, value: partition,
label: partition.toString(), label: `Partition #${partition.toString()}`,
}; };
} }
@ -64,6 +64,6 @@ export const getSelectedPartitionsFromSeekToParam = (
return partitions.map(({ partition }) => ({ return partitions.map(({ partition }) => ({
value: partition, value: partition,
label: partition.toString(), label: `Partition #${partition.toString()}`,
})); }));
}; };

4
kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/__test__/utils.spec.ts
View file

@ -101,7 +101,7 @@ describe('utils', () => {
it('returns parsed partition from params when partition list includes selected partition', () => { it('returns parsed partition from params when partition list includes selected partition', () => {
searchParams.set('seekTo', '42::0'); searchParams.set('seekTo', '42::0');
expect(getSelectedPartitionsFromSeekToParam(searchParams, part)).toEqual([ expect(getSelectedPartitionsFromSeekToParam(searchParams, part)).toEqual([
{ label: '42', value: 42 }, { label: 'Partition #42', value: 42 },
]); ]);
}); });
it('returns parsed partition from params when partition list NOT includes selected partition', () => { it('returns parsed partition from params when partition list NOT includes selected partition', () => {
@ -113,7 +113,7 @@ describe('utils', () => {
it('returns partitions when param "seekTo" is not defined', () => { it('returns partitions when param "seekTo" is not defined', () => {
searchParams.delete('seekTo'); searchParams.delete('seekTo');
expect(getSelectedPartitionsFromSeekToParam(searchParams, part)).toEqual([ expect(getSelectedPartitionsFromSeekToParam(searchParams, part)).toEqual([
{ label: '42', value: 42 }, { label: 'Partition #42', value: 42 },
]); ]);
}); });
}); });