Senghoung/edit review feature #189
Annotations
4 errors and 10 warnings
Lint:
site/src/component/Review/SubReview.tsx#L100
Visible, non-interactive elements with click handlers must have at least one keyboard listener
|
Lint:
site/src/component/Review/SubReview.tsx#L100
Avoid non-native interactive elements. If using native HTML is not possible, add an appropriate role and support for tabbing, mouse, keyboard, and touch inputs to an interactive content element
|
Lint:
site/src/component/UserReviews/UserReviews.tsx#L92
The autoFocus prop should not be used, as it can reduce usability and accessibility for users
|
Lint
Process completed with exit code 1.
|
Lint:
site/src/component/GradeDist/GradeDist.tsx#L70
React Hook useEffect has a missing dependency: 'fetchGradeDistData'. Either include it or remove the dependency array
|
Lint:
site/src/component/GradeDist/GradeDist.tsx#L81
React Hook useEffect has missing dependencies: 'createCourseEntries', 'createProfEntries', 'props.course', and 'props.professor'. Either include them or remove the dependency array
|
Lint:
site/src/component/GradeDist/GradeDist.tsx#L88
React Hook useEffect has missing dependencies: 'createQuarterEntries' and 'gradeDistData.length'. Either include them or remove the dependency array
|
Lint:
site/src/component/Report/ReportGroup.tsx#L21
Unexpected console statement
|
Lint:
site/src/component/Report/ReportGroup.tsx#L27
React Hook useEffect has a missing dependency: 'props.reviewID'. Either include it or remove the dependency array
|
Lint:
site/src/component/Report/Reports.tsx#L46
React Hook useEffect has a missing dependency: 'getData'. Either include it or remove the dependency array
|
Lint:
site/src/component/Review/Review.tsx#L95
React Hook useEffect has missing dependencies: 'dispatch' and 'getReviews'. Either include them or remove the dependency array
|
Lint:
site/src/component/ReviewForm/ReviewForm.tsx#L108
React Hook useEffect has missing dependencies: 'cookies.user' and 'props'. 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
|
Lint:
site/src/component/Schedule/Schedule.tsx#L31
React Hook useEffect has a missing dependency: 'fetchScheduleDataFromAPI'. Either include it or remove the dependency array
|
Lint:
site/src/component/SideInfo/SideInfo.tsx#L138
React Hook useEffect has missing dependencies: 'allToken' and 'props.searchType'. Either include them or remove the dependency array
|
The logs for this run have expired and are no longer available.
Loading