Skip to content

release: navigation, nettoyage du code mort et consolidation de la documentation - #304

Merged
horacioskrp merged 6 commits into
mainfrom
develop
Sep 2, 2026
Merged

release: navigation, nettoyage du code mort et consolidation de la documentation#304
horacioskrp merged 6 commits into
mainfrom
develop

Conversation

@horacioskrp

Copy link
Copy Markdown
Collaborator

Publication de develop vers main — 4 commits.

Navigation (2 commits)

Le menu perdait son état actif dès qu'on ouvrait une sous-page (/students/create, /students/{id}/edit…), et le fil d'Ariane disparaissait avec lui. isCurrentUrl faisait une égalité stricte sur le chemin.

Corrigé à la racine, dans le hook useCurrentUrl, pour les 4 menus (sidebar, fil d'Ariane, sous-menu Paramètres, en-tête). Deux règles évitent les faux positifs :

  • par segment/students ne capte pas /students-import ni /students-stats ;
  • le plus spécifique gagne — sur /accounting/transactions, seul « Journal des transactions » s'allume.

« Mon compte » a nécessité un champ match?: string[] sur NavItem : ses sections (mot de passe, 2FA, apparence) n'ont pas de préfixe commun exclusif, et élargir à /settings aurait empiété sur Paramètres.

Vérifié dans le navigateur sur 8 pages, cas pièges compris.

Nettoyage du code mort

12 composants/layouts supprimés (aucun importé), dont deux cas de mort transitive : app-header (importé seulement par app-header-layout, jamais monté) et toggle (référencé seulement par toggle-group, orphelin). ui/skeleton paraissait orphelin mais sert à ui/sidebar — conservé.

2 dépendances retirées : @radix-ui/react-toggle, @radix-ui/react-toggle-group.

Documentation consolidée

docs/ reste dédié à l'API ; la doc technique est regroupée et structurée dans documentations/ (prise-en-main, administration, exploitation, architecture) avec un index.

Dédoublonnage : 4 documents matricule → 1, 2 documents permissions → 1 (guide enrichi des garde-fous issus de l'audit). Lien mort du README corrigé.

Bilan

32 fichiers, +327 / −2639 lignes — l'essentiel étant de la redondance retirée.

Débloque aussi les liens de la documentation publique (https://dalibi.wearekarfi.dev/documentation), qui pointent vers la nouvelle arborescence.

@horacioskrp
horacioskrp merged commit 5369c29 into main Sep 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant