maximus/Simpl-Resultat
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases 19 Packages Wiki Activity Actions

[#1] Fondation non-breaking : suppression pivot + squelettes + shared components #69

New issue
Closed
opened 2026-04-13 14:58:54 +00:00 by maximus · 0 comments
maximus commented 2026-04-13 14:58:54 +00:00
Owner
Copy link

Objectif

Poser la fondation de la refonte des rapports sans rien casser : supprimer franchement le pivot, créer les 4 squelettes de pages plates, ajouter les sous-routes, créer les shared components (ViewModeToggle, Sparkline, ContextMenu générique). Les 4 rapports existants (trends, byCategory, overTime, budgetVsActual) continuent de fonctionner comme avant.

Spec : spec-refonte-rapports.md

Tâches

Suppression franche du pivot

  • Delete src/components/reports/DynamicReport.tsx
  • Delete src/components/reports/DynamicReportPanel.tsx
  • Delete src/components/reports/DynamicReportTable.tsx
  • Delete src/components/reports/DynamicReportChart.tsx
  • Retirer de src/hooks/useReports.ts : les champs pivotConfig, pivotResult, les actions setPivotConfig, la branche tab === 'dynamic' dans le reducer et les fetches
  • Nettoyer le type ReportTab dans src/shared/types.ts (plus de 'dynamic')
  • Retirer l'onglet pivot de src/pages/ReportsPage.tsx (constante TABS, imports, case dans le switch)
  • Supprimer toutes les clés reports.pivot.* dans src/i18n/locales/fr.json et en.json

Squelettes des nouvelles pages (plat dans src/pages/)

  • Créer src/pages/ReportsHighlightsPage.tsx — placeholder {t("common.underConstruction")}
  • Créer src/pages/ReportsTrendsPage.tsx — placeholder
  • Créer src/pages/ReportsComparePage.tsx — placeholder
  • Créer src/pages/ReportsCategoryPage.tsx — placeholder
  • Ajouter les 4 sous-routes dans src/App.tsx : /reports/highlights, /reports/trends, /reports/compare, /reports/category

Shared components

  • Créer src/components/reports/ViewModeToggle.tsx — API { value: 'chart' | 'table', onChange, storageKey }, persiste dans localStorage
  • Créer src/components/reports/Sparkline.tsx — Recharts LineChart compact sans axes
  • Créer src/components/shared/ContextMenu.tsx — shell générique { x, y, items: { label, onClick }[], onClose } avec click-outside + Escape
  • Refactorer src/components/shared/ChartContextMenu.tsx pour composer ContextMenu (garder ses props spécifiques aux charts : categoryName, onHide, onViewDetails)

i18n

  • Ajouter en FR et EN : reports.hub.*, reports.viewMode.*, reports.empty.*, common.underConstruction
  • Vérifier la parité FR/EN des clés ajoutées/retirées

Nav

  • NAV_ITEMS dans src/shared/constants/index.ts : aucun changement — l'entrée /reports reste seule, les sous-rapports ne sont accessibles que via les cartes du hub (Issue #3)

Vérifications

  • npm run build vert
  • cargo check vert
  • npm test vert
  • grep -ri "DynamicReport\|pivotConfig\|reports\.pivot" src/ retourne vide
  • Les 4 rapports existants tournent toujours (non-regression visuelle sur /reports)

Critères d'acceptation

  • L'app compile sans erreurs TypeScript
  • Les 4 nouvelles routes affichent leur placeholder sans planter
  • Le pivot est totalement absent du code, des types et des traductions
  • Les 4 rapports restants ne régressent pas
  • Aucun shared component n'est cassé (ChartContextMenu fonctionne toujours là où il est utilisé aujourd'hui)
## Objectif Poser la fondation de la refonte des rapports **sans rien casser** : supprimer franchement le pivot, créer les 4 squelettes de pages plates, ajouter les sous-routes, créer les shared components (`ViewModeToggle`, `Sparkline`, `ContextMenu` générique). Les 4 rapports existants (trends, byCategory, overTime, budgetVsActual) continuent de fonctionner comme avant. Spec : `spec-refonte-rapports.md` ## Tâches ### Suppression franche du pivot - [ ] Delete `src/components/reports/DynamicReport.tsx` - [ ] Delete `src/components/reports/DynamicReportPanel.tsx` - [ ] Delete `src/components/reports/DynamicReportTable.tsx` - [ ] Delete `src/components/reports/DynamicReportChart.tsx` - [ ] Retirer de `src/hooks/useReports.ts` : les champs `pivotConfig`, `pivotResult`, les actions `setPivotConfig`, la branche `tab === 'dynamic'` dans le reducer et les fetches - [ ] Nettoyer le type `ReportTab` dans `src/shared/types.ts` (plus de `'dynamic'`) - [ ] Retirer l'onglet pivot de `src/pages/ReportsPage.tsx` (constante `TABS`, imports, case dans le switch) - [ ] Supprimer toutes les clés `reports.pivot.*` dans `src/i18n/locales/fr.json` **et** `en.json` ### Squelettes des nouvelles pages (plat dans `src/pages/`) - [ ] Créer `src/pages/ReportsHighlightsPage.tsx` — placeholder `{t("common.underConstruction")}` - [ ] Créer `src/pages/ReportsTrendsPage.tsx` — placeholder - [ ] Créer `src/pages/ReportsComparePage.tsx` — placeholder - [ ] Créer `src/pages/ReportsCategoryPage.tsx` — placeholder - [ ] Ajouter les 4 sous-routes dans `src/App.tsx` : `/reports/highlights`, `/reports/trends`, `/reports/compare`, `/reports/category` ### Shared components - [ ] Créer `src/components/reports/ViewModeToggle.tsx` — API `{ value: 'chart' | 'table', onChange, storageKey }`, persiste dans localStorage - [ ] Créer `src/components/reports/Sparkline.tsx` — Recharts `LineChart` compact sans axes - [ ] Créer `src/components/shared/ContextMenu.tsx` — shell générique `{ x, y, items: { label, onClick }[], onClose }` avec click-outside + Escape - [ ] Refactorer `src/components/shared/ChartContextMenu.tsx` pour composer `ContextMenu` (garder ses props spécifiques aux charts : `categoryName`, `onHide`, `onViewDetails`) ### i18n - [ ] Ajouter en FR et EN : `reports.hub.*`, `reports.viewMode.*`, `reports.empty.*`, `common.underConstruction` - [ ] Vérifier la parité FR/EN des clés ajoutées/retirées ### Nav - [ ] `NAV_ITEMS` dans `src/shared/constants/index.ts` : **aucun changement** — l'entrée `/reports` reste seule, les sous-rapports ne sont accessibles que via les cartes du hub (Issue #3) ### Vérifications - [ ] `npm run build` vert - [ ] `cargo check` vert - [ ] `npm test` vert - [ ] `grep -ri "DynamicReport\|pivotConfig\|reports\.pivot" src/` retourne vide - [ ] Les 4 rapports existants tournent toujours (non-regression visuelle sur `/reports`) ## Critères d'acceptation - L'app compile sans erreurs TypeScript - Les 4 nouvelles routes affichent leur placeholder sans planter - Le pivot est totalement absent du code, des types et des traductions - Les 4 rapports restants ne régressent pas - Aucun shared component n'est cassé (`ChartContextMenu` fonctionne toujours là où il est utilisé aujourd'hui)
maximus added this to the spec-refonte-rapports milestone 2026-04-13 14:58:54 +00:00
maximus added the
status:ready
type:refactor
source:human
 labels 2026-04-13 14:58:54 +00:00
maximus added
status:approved
 and removed
status:ready
 labels 2026-04-14 18:26:36 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
issue-67-feedback-widget
issue-97-cartes-page
issue-96-compare-refactor
issue-51-compte-maximus-oauth
issue-46-license-commands-entitlements
issue-48-gate-auto-updates
issue-47-license-ui-card
fix/simpl-resultat-54-argon2id-pin
fix/simpl-resultat-59-bump-vite
issue-43-update-picomatch
fix/simpl-resultat-41-category-time-report-filter
fix/simpl-resultat-31-dashboard-month-dropdown
fix/simpl-resultat-29-budget-visual-adjustments
fix/simpl-resultat-23-dashboard-category-ordering
fix/simpl-resultat-16-prev-year-budget
fix/simpl-resultat-19-tooltip-transparency
fix/simpl-resultat-17-remove-piechart-title
fix/simpl-resultat-16-budget-previous-year
v0.8.2
v0.8.1
v0.8.0
v0.7.4
v0.7.3
v0.7.2
v0.7.1
v0.7.0
v0.6.7
v0.6.6
v0.6.5
v0.6.4
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.2
v0.5.0
v0.4.2
v0.4.0
v0.3.11
v0.3.10
v0.3.9
v0.3.8
v0.3.7
v0.3.6
v0.3.5
v0.3.4
v0.3.3
v0.3.2
v0.3.1
v0.3.0
v0.2.12
v0.2.11
v0.2.10
v0.2.9
v0.2.8
v0.2.7
v0.2.6
v0.2.5
v0.2.4
v0.2.3
v0.2.2
v0.2.1
v0.2.0
v0.1.2
v0.1.1
v0.1.0
Labels
Clear labels   
source:analyste

Créée par le Super-Analyste

  
source:defenseur

Créée par l Escouade Défenseur

  
source:human

Créée par un humain

  
source:medic

Créée par le Medic (auto-détection)

  
status:approved

PR approuvée, prête à merger

  
status:blocked

Bloquée (question, dépendance, clarification)

  
status:in-progress

Un agent (Medic/Dev) travaille dessus

  
status:needs-fix

Reviewer demande des corrections

  
status:ready

Analysée et documentée, prête à prendre

  
status:review

PR créée, en attente du Reviewer

  
status:triage

Nouvelle issue, pas encore analysée

  
type:bug

Correction de bug → Medic

  
type:feature

Nouvelle fonctionnalité → Dev

  
type:infra

Config, CI/CD, déploiement → Dev

  
type:refactor

Refactoring / amélioration technique → Dev

  
type:schema

Modèle de données / migrations → Dev

  
type:security

Issue de sécurité → Medic ou Dev

No labels
source:analyste
source:defenseur
source:human
source:medic
status:approved
status:blocked
status:in-progress
status:needs-fix
status:ready
status:review
status:triage
type:bug
type:feature
type:infra
type:refactor
type:schema
type:security
Milestone
Clear milestone
No items
No milestone
spec-refonte-rapports
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: maximus/Simpl-Resultat#69
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?