[Class statistics] Fix page crash when opening courses tab before dat… #1632
joonas-astaging.yaml
on: push
test
/
Check if new migrations have been added
6s
test
/
Lint backend and frontend
1m 29s
test
/
Build anon sis-db and push to toska hub if new migrations have been added
0s
test
/
Build anon kone-db and push to toska hub if new migrations have been added
0s
test
/
Build anon user-db and push to toska hub if new migrations have been added
0s
Matrix: test / test_end_to_end
Build and release frontend
0s
Build and release backend
0s
Report deployment success to the "oodikone_notifications" channel
0s
Report deployment failure to the "oodikone_notifications" channel
4s
Annotations
1 error and 10 warnings
|
test / Test filters
Cypress tests: 1 failed
|
|
test / Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/FacultiesTab/index.jsx#L39
React Hook useMemo has a missing dependency: 'selectedSemesters'. Either include it or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/FacultiesTab/index.jsx#L65
React Hook useMemo has a missing dependency: 'totalRow'. Either include it or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/index.jsx#L51
React Hook useEffect has a missing dependency: 'semesterFilter'. Either include it or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/selectorComponents.jsx#L76
React Hook useMemo has a missing dependency: 'getTextIn'. Either include it or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L37
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L44
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L51
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L84
React Hook useEffect has missing dependencies: 'parseQueryFromUrl' and 'setValues'. Either include them or remove the dependency array. If 'setValues' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
|
test / Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/OpenUniPopulationResults/index.jsx#L232
React Hook useEffect has missing dependencies: 'getTextIn', 'isError', and 'isFetchingOrLoading'. Either include them or remove the dependency array
|
|
test / Lint backend and frontend:
services/frontend/src/components/CustomPopulation/index.jsx#L44
The 'allSemesters' logical expression could make the dependencies of useMemo Hook (at line 84) change on every render. Move it inside the useMemo callback. Alternatively, wrap the initialization of 'allSemesters' in its own useMemo() Hook
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
Cypress videos - Test filters
|
38.8 MB |
|
|
backend-image
|
115 MB |
|