chore(front): extrait KpiCard, unifie la clé de session, retire du code mort
- KpiCard était déclaré dans Dashboard.jsx : sorti dans son propre fichier avec ses styles, les règles CSS devenues orphelines sont supprimées de Dashboard.module.css. - La clé localStorage 'ds_user' était définie séparément dans api.js (USER_KEY) et AuthContext.jsx (STORAGE_KEY). AuthContext importe désormais la constante exportée par api.js : un renommage ne peut plus casser silencieusement la restauration de session. - hooks/useApi.js n'était importé nulle part : supprimé.
This commit is contained in:
@@ -0,0 +1,10 @@
|
|||||||
|
import styles from './KpiCard.module.css';
|
||||||
|
|
||||||
|
export default function KpiCard({ label, value, color }) {
|
||||||
|
return (
|
||||||
|
<div className={styles.kpiCard}>
|
||||||
|
<span className={styles.kpiValue} style={{ color }}>{value}</span>
|
||||||
|
<span className={styles.kpiLabel}>{label}</span>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -0,0 +1,28 @@
|
|||||||
|
.kpiCard {
|
||||||
|
display: flex;
|
||||||
|
flex-direction: column;
|
||||||
|
align-items: center;
|
||||||
|
padding: 20px;
|
||||||
|
background: var(--color-white);
|
||||||
|
border-radius: var(--radius);
|
||||||
|
box-shadow: var(--shadow);
|
||||||
|
}
|
||||||
|
|
||||||
|
.kpiValue {
|
||||||
|
font-size: 36px;
|
||||||
|
font-weight: 700;
|
||||||
|
line-height: 1;
|
||||||
|
margin-bottom: 6px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.kpiLabel {
|
||||||
|
font-size: 12px;
|
||||||
|
color: var(--color-muted, #888);
|
||||||
|
text-align: center;
|
||||||
|
}
|
||||||
|
|
||||||
|
@media (max-width: 768px) {
|
||||||
|
.kpiValue {
|
||||||
|
font-size: 28px;
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -2,16 +2,15 @@
|
|||||||
|
|
||||||
import { createContext, useContext, useState } from 'react';
|
import { createContext, useContext, useState } from 'react';
|
||||||
import * as api from '../services/api';
|
import * as api from '../services/api';
|
||||||
|
import { USER_KEY } from '../services/api';
|
||||||
|
|
||||||
const AuthContext = createContext(null);
|
const AuthContext = createContext(null);
|
||||||
|
|
||||||
const STORAGE_KEY = 'ds_user';
|
|
||||||
|
|
||||||
/* Fournisseur du contexte d'authentification */
|
/* Fournisseur du contexte d'authentification */
|
||||||
export function AuthProvider({ children }) {
|
export function AuthProvider({ children }) {
|
||||||
const [user, setUser] = useState(() => {
|
const [user, setUser] = useState(() => {
|
||||||
/* Restauration de la session depuis le localStorage au montage */
|
/* Restauration de la session depuis le localStorage au montage */
|
||||||
const stored = localStorage.getItem(STORAGE_KEY);
|
const stored = localStorage.getItem(USER_KEY);
|
||||||
return stored ? JSON.parse(stored) : null;
|
return stored ? JSON.parse(stored) : null;
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -19,7 +18,7 @@ export function AuthProvider({ children }) {
|
|||||||
async function login(username, password) {
|
async function login(username, password) {
|
||||||
const connectedUser = await api.login(username, password);
|
const connectedUser = await api.login(username, password);
|
||||||
setUser(connectedUser);
|
setUser(connectedUser);
|
||||||
localStorage.setItem(STORAGE_KEY, JSON.stringify(connectedUser));
|
localStorage.setItem(USER_KEY, JSON.stringify(connectedUser));
|
||||||
return connectedUser;
|
return connectedUser;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -27,7 +26,7 @@ export function AuthProvider({ children }) {
|
|||||||
function logout() {
|
function logout() {
|
||||||
api.clearToken();
|
api.clearToken();
|
||||||
setUser(null);
|
setUser(null);
|
||||||
localStorage.removeItem(STORAGE_KEY);
|
localStorage.removeItem(USER_KEY);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Booléen pratique pour savoir si l'utilisateur est connecté */
|
/* Booléen pratique pour savoir si l'utilisateur est connecté */
|
||||||
|
|||||||
@@ -1,33 +0,0 @@
|
|||||||
/* Hook générique pour les appels API : retourne { data, loading, error, refetch } */
|
|
||||||
|
|
||||||
import { useState, useEffect, useCallback } from 'react';
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @param {Function} fetchFn - Fonction async à appeler (depuis api.js)
|
|
||||||
* @param {Array} deps - Dépendances qui déclenchent un re-fetch (optionnel)
|
|
||||||
*/
|
|
||||||
export function useApi(fetchFn, deps = []) {
|
|
||||||
const [data, setData] = useState(null);
|
|
||||||
const [loading, setLoading] = useState(true);
|
|
||||||
const [error, setError] = useState(null);
|
|
||||||
|
|
||||||
const fetch = useCallback(async () => {
|
|
||||||
setLoading(true);
|
|
||||||
setError(null);
|
|
||||||
try {
|
|
||||||
const result = await fetchFn();
|
|
||||||
setData(result);
|
|
||||||
} catch (err) {
|
|
||||||
setError(err.message || 'Une erreur est survenue.');
|
|
||||||
} finally {
|
|
||||||
setLoading(false);
|
|
||||||
}
|
|
||||||
// eslint-disable-next-line react-hooks/exhaustive-deps
|
|
||||||
}, deps);
|
|
||||||
|
|
||||||
useEffect(() => {
|
|
||||||
fetch();
|
|
||||||
}, [fetch]);
|
|
||||||
|
|
||||||
return { data, loading, error, refetch: fetch };
|
|
||||||
}
|
|
||||||
@@ -4,20 +4,11 @@ import { useState, useEffect, useCallback, useMemo } from 'react';
|
|||||||
import { RefreshCw } from 'lucide-react';
|
import { RefreshCw } from 'lucide-react';
|
||||||
import { getDashboard, getDashboardSummary, getDashboardFiltres } from '../../services/api';
|
import { getDashboard, getDashboardSummary, getDashboardFiltres } from '../../services/api';
|
||||||
import MonitoringCard from '../../components/common/MonitoringCard';
|
import MonitoringCard from '../../components/common/MonitoringCard';
|
||||||
|
import KpiCard from '../../components/common/KpiCard';
|
||||||
import LoadingSpinner from '../../components/common/LoadingSpinner';
|
import LoadingSpinner from '../../components/common/LoadingSpinner';
|
||||||
import ErrorMessage from '../../components/common/ErrorMessage';
|
import ErrorMessage from '../../components/common/ErrorMessage';
|
||||||
import styles from './Dashboard.module.css';
|
import styles from './Dashboard.module.css';
|
||||||
|
|
||||||
/* Carte KPI affichant un indicateur clé */
|
|
||||||
function KpiCard({ label, value, color }) {
|
|
||||||
return (
|
|
||||||
<div className={styles.kpiCard}>
|
|
||||||
<span className={styles.kpiValue} style={{ color }}>{value}</span>
|
|
||||||
<span className={styles.kpiLabel}>{label}</span>
|
|
||||||
</div>
|
|
||||||
);
|
|
||||||
}
|
|
||||||
|
|
||||||
export default function Dashboard() {
|
export default function Dashboard() {
|
||||||
const [summary, setSummary] = useState(null);
|
const [summary, setSummary] = useState(null);
|
||||||
const [monitorings, setMonitorings] = useState([]);
|
const [monitorings, setMonitorings] = useState([]);
|
||||||
|
|||||||
@@ -55,29 +55,6 @@
|
|||||||
margin-bottom: 32px;
|
margin-bottom: 32px;
|
||||||
}
|
}
|
||||||
|
|
||||||
.kpiCard {
|
|
||||||
display: flex;
|
|
||||||
flex-direction: column;
|
|
||||||
align-items: center;
|
|
||||||
padding: 20px;
|
|
||||||
background: var(--color-white);
|
|
||||||
border-radius: var(--radius);
|
|
||||||
box-shadow: var(--shadow);
|
|
||||||
}
|
|
||||||
|
|
||||||
.kpiValue {
|
|
||||||
font-size: 36px;
|
|
||||||
font-weight: 700;
|
|
||||||
line-height: 1;
|
|
||||||
margin-bottom: 6px;
|
|
||||||
}
|
|
||||||
|
|
||||||
.kpiLabel {
|
|
||||||
font-size: 12px;
|
|
||||||
color: var(--color-muted, #888);
|
|
||||||
text-align: center;
|
|
||||||
}
|
|
||||||
|
|
||||||
.filtres {
|
.filtres {
|
||||||
display: flex;
|
display: flex;
|
||||||
gap: 12px;
|
gap: 12px;
|
||||||
@@ -131,10 +108,6 @@
|
|||||||
gap: 10px;
|
gap: 10px;
|
||||||
}
|
}
|
||||||
|
|
||||||
.kpiValue {
|
|
||||||
font-size: 28px;
|
|
||||||
}
|
|
||||||
|
|
||||||
.filtres {
|
.filtres {
|
||||||
flex-direction: column;
|
flex-direction: column;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user