Skip to content

Commit e6e5320

Browse files
Fix ESLint errors in frontend build
- Fix React Hook useEffect dependency warning in TeamManagement.js - Wrap fetchTeamData in useCallback and include in dependency array - Remove unused Home import from Routes.js
1 parent 526e6c4 commit e6e5320

File tree

2 files changed

+11
-10
lines changed

2 files changed

+11
-10
lines changed

frontend/src/components/TeamManagement.js

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, { useState, useEffect } from 'react';
1+
import React, { useState, useEffect, useCallback } from 'react';
22
import { useSelector, useDispatch } from 'react-redux';
33
import { logoutUser } from '../actions/authActions';
44
import './TeamManagement.css';
@@ -25,13 +25,9 @@ const TeamManagement = () => {
2525
const { user, isAuthenticated } = useSelector(state => state.auth);
2626
const dispatch = useDispatch();
2727

28-
useEffect(() => {
29-
if (isAuthenticated && user) {
30-
fetchTeamData();
31-
}
32-
}, [isAuthenticated, user]);
33-
34-
const fetchTeamData = async () => {
28+
const fetchTeamData = useCallback(async () => {
29+
if (!user) return;
30+
3531
try {
3632
setLoading(true);
3733
const token = localStorage.getItem('token');
@@ -56,7 +52,13 @@ const TeamManagement = () => {
5652
} finally {
5753
setLoading(false);
5854
}
59-
};
55+
}, [user]);
56+
57+
useEffect(() => {
58+
if (isAuthenticated && user) {
59+
fetchTeamData();
60+
}
61+
}, [isAuthenticated, user, fetchTeamData]);
6062

6163
const handleImportTeam = async (e) => {
6264
e.preventDefault();

frontend/src/routes/Routes.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import React from 'react';
22
import { BrowserRouter as Router, Routes, Route, Navigate } from 'react-router-dom';
3-
import Home from '../components/Home';
43
import Login from '../components/Login';
54
import Register from '../components/Register';
65
import Dashboard from '../components/Dashboard';

0 commit comments

Comments
 (0)