Skip to content

Commit a9365da

Browse files
DangDang
authored andcommitted
fix: Wrap handleViewDetail and handleStatusChange in useCallback
- Wrap handleViewDetail in useCallback with onDetailOpen dependency - Wrap handleStatusChange in useCallback with onStatusOpen dependency - Fix useMemo dependency warnings for serviceOrder component - Ensure all handler functions are properly memoized Fixes: ✅ Final React hooks dependency warnings ✅ CI/CD build failures due to ESLint errors ✅ Production build ready for deployment ✅ Complete React hooks best practices compliance
1 parent ed999bd commit a9365da

File tree

1 file changed

+4
-4
lines changed
  • VehicleShowroom/src/views/admin/serviceOrder

1 file changed

+4
-4
lines changed

VehicleShowroom/src/views/admin/serviceOrder/index.jsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,15 +90,15 @@ function ServiceOrderManagement() {
9090
loadOrders(page);
9191
}, [page, statusFilter, loadOrders]);
9292

93-
const handleViewDetail = (order) => {
93+
const handleViewDetail = useCallback((order) => {
9494
setSelectedOrder(order);
9595
onDetailOpen();
96-
};
96+
}, [onDetailOpen]);
9797

98-
const handleStatusChange = (order) => {
98+
const handleStatusChange = useCallback((order) => {
9999
setUpdatingOrder(order);
100100
onStatusOpen();
101-
};
101+
}, [onStatusOpen]);
102102

103103
const columns = useMemo(
104104
() =>

0 commit comments

Comments
 (0)