Merge pull request #119 from guanquann/test-fix · CS3219-AY2425S1/cs3219-ay2425s1-project-g28@1342ab4

Triggered via push November 10, 2024 05:45

jolynloh

development

Status Success

Total duration 1m 47s

Artifacts –

ci.yml

on: push

Matrix: backend-ci

3 warnings

frontend-ci: frontend/src/components/CollabSessionControls/index.tsx#L73

React Hook useEffect has a missing dependency: 'handleConfirmEndSession'. Either include it or remove the dependency array

frontend-ci: frontend/src/components/CollabSessionControls/index.tsx#L94

React Hook useEffect has a missing dependency: 'setStopTime'. Either include it or remove the dependency array

frontend-ci: frontend/src/pages/NewQuestion/index.tsx#L35

Fast refresh only works when a file only exports components. Use a new file to share constants or functions between components