Skip to content

Commit ae854bf

Browse files
committed
Merge branch 'dev' into project-to-workspace
2 parents 9fc6cc1 + 5919829 commit ae854bf

File tree

24 files changed

+103
-72
lines changed

24 files changed

+103
-72
lines changed

src/ui/components/Filters/index.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import {
99
Paragraph,
1010
} from '../../components';
1111
import { iconColors, iconSizes } from '../../../constants';
12-
import { formatDateToDisplay } from '../../../utils';
12+
import { formatDateToDisplayWithoutTime } from '../../../utils';
1313
import DatePicker from 'react-datepicker';
1414

1515
import 'bootstrap/dist/css/bootstrap.min.css';
@@ -736,6 +736,7 @@ const FilterComponent = ({
736736
function getSecondColumnOptions(options: any, type: any) {
737737
return options.filter((o: any) => o.type === type);
738738
}
739+
const validFilters = filters?.filter((item: any) => item.filterValue);
739740

740741
return (
741742
<FlexBox.Column fullWidth>
@@ -784,7 +785,7 @@ const FilterComponent = ({
784785
{console.log(filters, 'filters1')}
785786
</Paragraph>
786787
) : filters[0]?.filterValue && !applyFilter && !searchText ? (
787-
filters.map((filter: any, index: number) => {
788+
validFilters.map((filter: any, index: number) => {
788789
return (
789790
<FlexBox.Row key={index} className={styles.tile}>
790791
<Box
@@ -812,7 +813,7 @@ const FilterComponent = ({
812813
filter.column.selectedValue.label,
813814
filter.filterValue,
814815
)
815-
: formatDateToDisplay(filter.filterValue)
816+
: formatDateToDisplayWithoutTime(filter.filterValue)
816817
}`}
817818
</Box>
818819

src/ui/layouts/common/Table/index.tsx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -284,13 +284,12 @@ export const Table: React.FC<TableProps> = ({
284284
if (fetchingMembers) {
285285
return <FullWidthSpinner color="black" size="md" />;
286286
}
287-
console.log(fetchingMembers, activeSorting, 'fetchingMembers');
287+
// console.log(fetchingMembers, activeSorting, 'fetchingMembers');
288288
const onChangePagePerItem = (p: number, size: number) => {
289289
// onChange(p + 1, size);
290290
setItemPerPage(size);
291291
};
292-
console.log('pages11', itemPerPage, ITEMS_PER_PAGE);
293-
292+
// console.log('pages11', itemPerPage, ITEMS_PER_PAGE);
294293
return (
295294
<FlexBox.Column fullWidth>
296295
<IfElse
@@ -585,7 +584,7 @@ export const Table: React.FC<TableProps> = ({
585584
</FlexBox>
586585
)}
587586
</If>
588-
{console.log(paginated, 'paginated')}
587+
{/* {console.log(paginated, 'paginated')} */}
589588
</>
590589
)}
591590
renderWhenFalse={() => (

src/ui/layouts/pipelines/PipelineDetail/Runs/index.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,20 @@ import { RunsTable } from '../../RunsTable';
44
import { useService } from './useService';
55

66
export const Runs: React.FC<{
7+
isExpended?: any;
78
fromAllruns?: boolean;
89
pipelineId: TId;
910
filter: any;
1011
runId?: any;
1112
pagination?: boolean;
12-
}> = ({ pipelineId, filter, pagination, runId, fromAllruns }) => {
13+
}> = ({ pipelineId, filter, pagination, runId, fromAllruns, isExpended }) => {
1314
const [sortBy, setSortBy] = useState('created');
1415
function getSorted(activeSorting: any, activeSortingDirection: any) {
1516
setSortBy(activeSortingDirection?.toLowerCase() + ':' + activeSorting);
1617
// console.log(activeSorting, activeSortingDirection, 'aaaaaaa');
1718
}
1819
const { fetching, runIds, runsPaginated } = useService({
20+
isExpended,
1921
sortBy,
2022
pipelineId,
2123
filter,

src/ui/layouts/pipelines/PipelineDetail/Runs/useService.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@ export const useService = ({
2121
sortBy,
2222
filter,
2323
pipelineId,
24+
isExpended,
2425
}: {
26+
isExpended?: any;
2527
filter: {
2628
column: filterValue;
2729
type: filterValue;
@@ -38,7 +40,7 @@ export const useService = ({
3840
console.log(runsPaginated, 'runsPaginated');
3941
useEffect(() => {}, [runs]);
4042
useEffect(() => {
41-
if (!isValidFilter) {
43+
if (!isValidFilter && !isExpended) {
4244
const intervalId = setInterval(() => {
4345
//assign interval to a variable to clear it.
4446

src/ui/layouts/pipelines/Pipelines/AllRuns/useService.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,9 @@ interface filterValue {
3535
export const useService = ({
3636
filter,
3737
sortBy,
38+
isExpended,
3839
}: {
40+
isExpended?: any;
3941
sortBy: string;
4042
filter: {
4143
column: filterValue;
@@ -48,9 +50,9 @@ export const useService = ({
4850
const runs = useSelector(runSelectors.myRuns);
4951
const selectedWorkspace = useSelector(workspaceSelectors.selectedWorkspace);
5052
const runsPaginated = useSelector(runSelectors.myRunsPaginated);
51-
const isValidFilter = filter.map((f) => f.value).join('');
53+
const isValidFilter = filter?.map((f) => f.value).join('');
5254
useEffect(() => {
53-
if (!isValidFilter) {
55+
if (!isValidFilter && !isExpended) {
5456
const intervalId = setInterval(() => {
5557
//assign interval to a variable to clear it.
5658
dispatch(

src/ui/layouts/pipelines/Pipelines/List/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ export const List: React.FC<Props> = ({
4141
activeSortingDirection,
4242
setActiveSortingDirection,
4343
setSelectedRunIds,
44-
} = useService(filter);
44+
} = useService({ filter, isExpended });
4545
const expendedRow = filteredPipelines.filter((item) => item.id === id);
4646

4747
const headerCols = GetHeaderCols({

src/ui/layouts/pipelines/Pipelines/List/useService.ts

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,17 @@ interface filterValue {
3535
type: string;
3636
value: string;
3737
}
38-
export const useService = (
38+
export const useService = ({
39+
filter,
40+
isExpended,
41+
}: {
42+
isExpended?: any;
3943
filter: {
4044
column: filterValue;
4145
type: filterValue;
4246
value: string;
43-
}[],
44-
): ServiceInterface => {
47+
}[];
48+
}): ServiceInterface => {
4549
const [activeSorting, setActiveSorting] = React.useState<Sorting | null>(
4650
'created',
4751
);
@@ -61,14 +65,14 @@ export const useService = (
6165
const pipelinesPaginated = useSelector(
6266
pipelineSelectors.myPipelinesPaginated,
6367
);
64-
const isValidFilter = filter.map((f) => f.value).join('');
68+
const isValidFilter = filter?.map((f) => f.value).join('');
6569
const { dispatchPipelineData } = callActionForPipelinesForPagination();
6670
useEffect(() => {
6771
setFilteredPipelines(pipelines as any[]);
6872
}, [filter, pipelines]);
6973

7074
useEffect(() => {
71-
if (!isValidFilter) {
75+
if (!isValidFilter && !isExpended) {
7276
const intervalId = setInterval(() => {
7377
const applySorting =
7478
activeSortingDirection?.toLowerCase() + ':' + activeSorting;

src/ui/layouts/pipelines/RunDetail/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ export const RunDetail: React.FC = () => {
124124
filter={[]}
125125
pagination={false}
126126
runId={runId}
127-
// isExpended
127+
isExpended
128128
pipelineId={pipelineId}
129129
></Runs>
130130
{/* <Box style={boxStyle}>

src/ui/layouts/pipelines/RunDetail/useService.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { useEffect, useState } from 'react';
22
import { RunDetailRouteParams } from '.';
3-
import { runPagesActions, runsActions } from '../../../../redux/actions';
4-
import { runPagesSelectors, runSelectors } from '../../../../redux/selectors';
3+
import { runsActions } from '../../../../redux/actions';
4+
import { runSelectors } from '../../../../redux/selectors';
55
import { useDispatch, useParams, useSelector } from '../../../hooks';
66

77
interface ServiceInterface {
@@ -15,7 +15,7 @@ export const useService = (): ServiceInterface => {
1515
const dispatch = useDispatch();
1616
const { id, pipelineId } = useParams<RunDetailRouteParams>();
1717
const [isMounted, setIsMounted] = useState(false);
18-
18+
const [fetching, setFetching] = useState(false);
1919
useEffect(() => {
2020
if (!isMounted) {
2121
setFetching(true);
@@ -42,10 +42,10 @@ export const useService = (): ServiceInterface => {
4242

4343
const run = useSelector(runSelectors.runForId(id));
4444

45-
const setFetching = (fetching: boolean) => {
46-
dispatch(runPagesActions.setFetching({ fetching }));
47-
};
48-
const fetching = useSelector(runPagesSelectors.fetching);
45+
// const setFetching = (fetching: boolean) => {
46+
// dispatch(runPagesActions.setFetching({ fetching }));
47+
// };
48+
// const fetching = useSelector(runPagesSelectors.fetching);
4949

5050
return { runId: id, pipelineId, run, fetching };
5151
};

src/ui/layouts/runs/RunDetail/index.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,7 @@ export const RunDetail: React.FC = () => {
9898
breadcrumbs={breadcrumbs}
9999
>
100100
<Runs
101+
isExpended
101102
filter={[]}
102103
pagination={false}
103104
runId={runId}

0 commit comments

Comments
 (0)