Skip to content

Set empty highlighted Item when remove a objetct #145

Set empty highlighted Item when remove a objetct

Set empty highlighted Item when remove a objetct #145

Triggered via pull request October 2, 2023 11:44
@Rub21Rub21
synchronize #26
build
Status Success
Total duration 3m 23s
Artifacts

checks.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

23 warnings
prep
The following actions uses node12 which is deprecated and will be forced to run on node16: styfle/[email protected], actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint: src/components/EncodeCanvas.js#L138
React Hook useEffect has missing dependencies: 'activeEncodeImageItem', 'map', 'requestSAM', and 'showNotification'. Either include them or remove the dependency array
lint: src/components/EncodeItem.js#L60
Redundant alt attribute. Screen-readers already announce `img` tags as an image. You don’t need to use the words `image`, `photo,` or `picture` (or any specified custom words) in the alt prop
lint: src/components/EncodeItems.js#L67
React Hook useEffect has a missing dependency: 'dispatchEncodeItems'. Either include it or remove the dependency array
lint: src/components/Header.js#L1
'useState' is defined but never used
lint: src/components/Items.js#L45
React Hook useLayoutEffect has a missing dependency: 'dispatchSetItems'. Either include it or remove the dependency array
lint: src/components/Projects.js#L43
React Hook useCallback has a missing dependency: 'dispatchActiveEncodeImageItem'. Either include it or remove the dependency array
lint: src/components/Projects.js#L66
React Hook useEffect has missing dependencies: 'projects.features' and 'setProject'. Either include them or remove the dependency array
lint: src/components/map/MagicWand.js#L58
React Hook useEffect has missing dependencies: 'activeClass.color', 'activeClass.name', 'activeProject.properties.name', 'dispatchSetItems', 'items', 'map', and 'wand'. Either include them or remove the dependency array
lint: src/components/map/ProjectLayer.js#L79
Assignments to the 'highlightedItem' variable from inside React Hook useEffect will be lost after each render. To preserve the value over time, store it in a useRef Hook and keep the mutable value in the '.current' property. Otherwise, you can move this variable directly inside useEffect
lint: src/components/map/ProjectLayer.js#L110
React Hook useEffect has missing dependencies: 'dispatchSetPointsSelector' and 'map'. Either include them or remove the dependency array
test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test: src/components/EncodeCanvas.js#L138
React Hook useEffect has missing dependencies: 'activeEncodeImageItem', 'map', 'requestSAM', and 'showNotification'. Either include them or remove the dependency array
test: src/components/EncodeItem.js#L60
Redundant alt attribute. Screen-readers already announce `img` tags as an image. You don’t need to use the words `image`, `photo,` or `picture` (or any specified custom words) in the alt prop
test: src/components/EncodeItems.js#L67
React Hook useEffect has a missing dependency: 'dispatchEncodeItems'. Either include it or remove the dependency array
test: src/components/Header.js#L1
'useState' is defined but never used
test: src/components/Items.js#L45
React Hook useLayoutEffect has a missing dependency: 'dispatchSetItems'. Either include it or remove the dependency array
test: src/components/Projects.js#L43
React Hook useCallback has a missing dependency: 'dispatchActiveEncodeImageItem'. Either include it or remove the dependency array
test: src/components/Projects.js#L66
React Hook useEffect has missing dependencies: 'projects.features' and 'setProject'. Either include them or remove the dependency array
test: src/components/map/MagicWand.js#L58
React Hook useEffect has missing dependencies: 'activeClass.color', 'activeClass.name', 'activeProject.properties.name', 'dispatchSetItems', 'items', 'map', and 'wand'. Either include them or remove the dependency array
test: src/components/map/ProjectLayer.js#L79
Assignments to the 'highlightedItem' variable from inside React Hook useEffect will be lost after each render. To preserve the value over time, store it in a useRef Hook and keep the mutable value in the '.current' property. Otherwise, you can move this variable directly inside useEffect
test: src/components/map/ProjectLayer.js#L110
React Hook useEffect has missing dependencies: 'dispatchSetPointsSelector' and 'map'. Either include them or remove the dependency array