[feat] Etendre useReportsPeriod : filtre compte (accountIds) + type ReportFilters #272

New issue

Closed

opened 2026-07-09 00:37:56 +00:00 by maximus · 0 comments

maximus commented

2026-07-09 00:37:56 +00:00

Owner

Etendre le hook partage useReportsPeriod avec le filtre compte (multi-comptes), de facon additive (pas de rename — 6 consommateurs en dependent). Ajoute accountIds: number[] URL-backed + le type ReportFilters. Fondation de la slice filtres de l'epic #260.

Fichiers concernes

src/hooks/useReportsPeriod.ts
src/shared/types/index.ts
test du hook

Depends on

(aucune)

Criteres d'acceptation

useReportsPeriod expose accountIds: number[] + setAccountIds, URL-backed (param query dedie, ex. sources)
Nouveau type ReportFilters { period, accountIds } dans shared/types
Parsing URL valide accountIds en entiers finis (rejette non-numerique), defaut [] (= aucun filtre)
Les 6 consommateurs (useTrends, useCompare, useCategoryZoom, ReportsCategoryPage, ReportsPage, ReportsComparePage) compilent inchanges (extension additive)
Tests unitaires : serialisation URL, defaut, validation entiers
build (tsc+vite) + npm test verts

Decisions prises

Multi-comptes (accountIds: number[]), pas singulier — arbitre apres analyse (debloque le sous-ensemble coherent, cas le plus pertinent pour un etat de resultat).
Hook additif, pas renomme : un rename casserait la compilation des 6 consommateurs et I1 doit etre vert seul.

Spec source

spec-plan-rapports-uniformes-suite.md

Etendre le hook partage `useReportsPeriod` avec le filtre compte (multi-comptes), de facon **additive** (pas de rename — 6 consommateurs en dependent). Ajoute `accountIds: number[]` URL-backed + le type `ReportFilters`. Fondation de la slice filtres de l'epic #260. ## Fichiers concernes - `src/hooks/useReportsPeriod.ts` - `src/shared/types/index.ts` - test du hook ## Depends on - (aucune) ## Criteres d'acceptation - [ ] `useReportsPeriod` expose `accountIds: number[]` + `setAccountIds`, URL-backed (param query dedie, ex. `sources`) - [ ] Nouveau type `ReportFilters { period, accountIds }` dans `shared/types` - [ ] Parsing URL valide `accountIds` en entiers finis (rejette non-numerique), defaut `[]` (= aucun filtre) - [ ] Les 6 consommateurs (`useTrends`, `useCompare`, `useCategoryZoom`, `ReportsCategoryPage`, `ReportsPage`, `ReportsComparePage`) compilent inchanges (extension additive) - [ ] Tests unitaires : serialisation URL, defaut, validation entiers - [ ] build (tsc+vite) + `npm test` verts ## Decisions prises - Multi-comptes (`accountIds: number[]`), pas singulier — arbitre apres analyse (debloque le sous-ensemble coherent, cas le plus pertinent pour un etat de resultat). - Hook **additif**, pas renomme : un rename casserait la compilation des 6 consommateurs et I1 doit etre vert seul. ## Spec source spec-plan-rapports-uniformes-suite.md