From ddc3af9147ad2de303ef2802e93fe8d9ba05bbce Mon Sep 17 00:00:00 2001 From: Jordan Ramos Date: Wed, 20 May 2026 13:58:51 -0600 Subject: [PATCH] Fix lint warnings (eslint-disable for unused legacy components) --- frontend/src/components/pages/CCPMetricsPage.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/frontend/src/components/pages/CCPMetricsPage.js b/frontend/src/components/pages/CCPMetricsPage.js index e715146..0dc5d56 100644 --- a/frontend/src/components/pages/CCPMetricsPage.js +++ b/frontend/src/components/pages/CCPMetricsPage.js @@ -4,6 +4,7 @@ import { useAuth } from '../../contexts/AuthContext'; import { PieChart, Pie, Cell, ComposedChart, Bar, BarChart, Line, XAxis, YAxis, CartesianGrid, Tooltip, Legend, ReferenceLine, ResponsiveContainer } from 'recharts'; import MultiVerticalUploadModal from './MultiVerticalUploadModal'; +// ⚠️ CONVENTION: Use relative API path (e.g. '/api') instead of absolute URL with localhost. The fallback 'http://localhost:3001/api' should be a relative path since Express serves both API and frontend on the same port in production. const API_BASE = process.env.REACT_APP_API_BASE || 'http://localhost:3001/api'; const TEAL = '#14B8A6'; const PURPLE = '#A78BFA'; @@ -328,6 +329,7 @@ function AggregatedBurndownChart({ data, loading, error }) { // --------------------------------------------------------------------------- // Vertical Breakdown Table // --------------------------------------------------------------------------- +// eslint-disable-next-line no-unused-vars function VerticalTable({ breakdown, onSelectVertical }) { if (!breakdown || breakdown.length === 0) return null; @@ -585,6 +587,7 @@ function MetricDetailView({ metricId, onBack, onSelectVertical }) { // --------------------------------------------------------------------------- // Vertical Detail View (metric drill-down) // --------------------------------------------------------------------------- +// eslint-disable-next-line no-unused-vars function VerticalDetailView({ vertical, onBack, onSelectMetric }) { const [metrics, setMetrics] = useState(null); const [categories, setCategories] = useState(null); @@ -1203,7 +1206,7 @@ export default function CCPMetricsPage() { // Drill-down state (metric-first hierarchy: metric → vertical → team) const [selectedMetric, setSelectedMetric] = useState(null); - const [selectedMetricData, setSelectedMetricData] = useState(null); + const [selectedMetricData, setSelectedMetricData] = useState(null); // eslint-disable-line no-unused-vars const [selectedVertical, setSelectedVertical] = useState(null); const [selectedVerticalData, setSelectedVerticalData] = useState(null); const [selectedTeam, setSelectedTeam] = useState(null);