Refacto details top icons #1986
dtrucsaction-intall-and-test.yml
on: pull_request
install-and-test
1m 44s
Annotations
10 warnings
|
install-and-test:
frontend/src/components/Header/Header.tsx#L32
Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
|
|
install-and-test:
frontend/src/components/ImageWithLegend/index.tsx#L36
Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
|
|
install-and-test:
frontend/src/components/Layout/useExternalScripts.tsx#L26
Unexpected any. Specify a different type
|
|
install-and-test:
frontend/src/components/Map/DetailsMap/DetailsMap.tsx#L164
React Hook useEffect has missing dependencies: 'props' and 'reportVisibility'. Either include them or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
|
|
install-and-test:
frontend/src/components/Map/components/HoverableMarker.tsx#L54
React Hook useMemo has a missing dependency: 'props'. Either include it or remove the dependency array
|
|
install-and-test:
frontend/src/components/Map/components/TileLayerExtended/index.tsx#L129
React Hook useEffect has missing dependencies: 'loadLayer' and 'rangeZoom'. Either include them or remove the dependency array
|
|
install-and-test:
frontend/src/components/Map/components/TileLayerExtended/index.tsx#L140
React Hook useEffect has a missing dependency: 'options?.attribution'. Either include it or remove the dependency array
|
|
install-and-test:
frontend/src/components/pages/details/Details.tsx#L709
React Hook useMemo has missing dependencies: 'anchors', 'handleViewPointClick', 'hasNavigator', 'sectionRef', 'sections', and 'setMapId'. Either include them or remove the dependency array
|
|
install-and-test:
frontend/src/components/pages/details/components/DetailsReservationWidget/DetailsReservationWidget.tsx#L60
React Hook useEffect has a missing dependency: 'isMounted'. Either include it or remove the dependency array
|
|
install-and-test:
frontend/src/components/pages/home/components/BannerCarousel/BannerCarousel.tsx#L24
Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
|