From: Lucas Di Pentima Date: Tue, 6 Jul 2021 19:51:48 +0000 (-0300) Subject: 17782: Fixes assorted compile warnings. X-Git-Tag: 2.2.2~29^2~6 X-Git-Url: https://git.arvados.org/arvados-workbench2.git/commitdiff_plain/110b716456d0cea66f8f526293f8bcaadaaeced4 17782: Fixes assorted compile warnings. There're some more not being addressed, the majority being from the cwl-svg library. Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima --- diff --git a/src/components/collection-panel-files/collection-panel-files.tsx b/src/components/collection-panel-files/collection-panel-files.tsx index 4667248c..41182482 100644 --- a/src/components/collection-panel-files/collection-panel-files.tsx +++ b/src/components/collection-panel-files/collection-panel-files.tsx @@ -75,7 +75,7 @@ export const CollectionPanelFilesComponent = ({ onItemMenuOpen, onSearchChange, useEffect(() => { onSearchChange(searchValue); - }, [searchValue]); + }, [onSearchChange, searchValue]); return ( { break; } case ProcessStatusFilter.CANCELLED: - case ProcessStatusFilter.FAILED: case ProcessStatusFilter.LOCKED: case ProcessStatusFilter.QUEUED: case ProcessStatusFilter.RUNNING: { diff --git a/src/store/resource-type-filters/resource-type-filters.ts b/src/store/resource-type-filters/resource-type-filters.ts index 8e902606..e42a16d8 100644 --- a/src/store/resource-type-filters/resource-type-filters.ts +++ b/src/store/resource-type-filters/resource-type-filters.ts @@ -290,7 +290,6 @@ export const buildProcessStatusFilters = ( fb:FilterBuilder, activeStatusFilter: break; } case ProcessStatusFilter.CANCELLED: - case ProcessStatusFilter.FAILED: case ProcessStatusFilter.LOCKED: case ProcessStatusFilter.RUNNING: { fb.addEqual('container.state', activeStatusFilter); diff --git a/src/store/virtual-machines/virtual-machines-actions.ts b/src/store/virtual-machines/virtual-machines-actions.ts index aa8cbfdd..e3b69d8d 100644 --- a/src/store/virtual-machines/virtual-machines-actions.ts +++ b/src/store/virtual-machines/virtual-machines-actions.ts @@ -81,7 +81,7 @@ export const loadVirtualMachinesUserData = () => export const saveRequestedDate = () => (dispatch: Dispatch, getState: () => RootState, services: ServiceRepository) => { - const date = formatDate((new Date).toISOString()); + const date = formatDate((new Date()).toISOString()); services.virtualMachineService.saveRequestedDate(date); dispatch(loadRequestedDate()); }; diff --git a/src/views-components/auto-logout/auto-logout.tsx b/src/views-components/auto-logout/auto-logout.tsx index 9a8299b2..a2d71d08 100644 --- a/src/views-components/auto-logout/auto-logout.tsx +++ b/src/views-components/auto-logout/auto-logout.tsx @@ -63,7 +63,7 @@ export const AutoLogoutComponent = (props: AutoLogoutProps) => { return () => { window.removeEventListener('storage', handleStorageEvents); }; - }, []); + }); const handleStorageEvents = (e: StorageEvent) => { if (e.key === LAST_ACTIVE_TIMESTAMP) { @@ -90,7 +90,7 @@ export const AutoLogoutComponent = (props: AutoLogoutProps) => { const handleOnAction = () => { // Notify the other tabs there was some activity. - const now = (new Date).getTime(); + const now = (new Date()).getTime(); localStorage.setItem(LAST_ACTIVE_TIMESTAMP, now.toString()); }; diff --git a/src/views/workflow-panel/workflow-description-card.tsx b/src/views/workflow-panel/workflow-description-card.tsx index 7da39b98..9c1d81c3 100644 --- a/src/views/workflow-panel/workflow-description-card.tsx +++ b/src/views/workflow-panel/workflow-description-card.tsx @@ -114,7 +114,7 @@ export const WorkflowDetailsCard = withStyles(styles)( return getWorkflowInputs(definition); } } - return; + return undefined; } renderInputsTable() {