jmartin/petitspas
1
0
Fork 0
Code Issues 28 Pull Requests Actions Packages Projects Releases Wiki Activity

[Cleanup] Renommer le préfixe Admin* du dashboard partagé (admin + gestionnaire) #155

New Issue
Closed
opened 2026-07-17 15:55:33 +00:00 by jmartin · 3 comments
jmartin commented 2026-07-17 15:55:33 +00:00
Owner
Copy Link
Copy Source

Contexte

Le panneau « Gestion des utilisateurs » est partagé admin + gestionnaire, mais une grande partie du code front est préfixée / rangée Admin* :

  • dossier frontend/lib/widgets/admin/
  • widgets : AdminChildDetailModal, AdminAmEditModal, AdminParentEditModal, AdminUserCard, AdminSelectAmModal, …
  • parfois dashboard_admin.dart

Le préfixe laisse croire que c’est réservé au rôle administrateur, alors que c’est du dashboard staff (admin et gestionnaire).

Est-ce que ça a du sens ?

Oui, pour la lisibilité et onboarding dev — mais c’est un rename mécanique large (imports partout), donc :

  • À faire quand la surface UI dashboard est un peu stabilisée (après vagues #132 / #153 / fiches), pour limiter les conflits de merge
  • Pas bloquant fonctionnellement pour la 0.1.0
  • Peut rester en 0.1.0 en fin de jalon, ou glisser en 0.2.0 si trop de churn

Proposition de naming

À trancher au kick-off du ticket :

Option Exemple
A — préfixe Staff StaffChildDetailModal
B — préfixe Dashboard / Gestion DashboardChildDetailModal
C — dossier neutre + noms sans préfixe widgets/dashboard/child_detail_modal.dart → ChildDetailModal

Reco : C (ou B) — évite un nouveau préfixe ambigu ; garder AdminManagementWidget seulement pour l’onglet Administrateurs.

Périmètre

  • Inventaire fichiers / classes Admin* du dashboard partagé
  • Décision naming
  • Rename + update imports
  • Vérifier routes / tests / docs qui citent les anciens chemins
  • Pas de changement comportemental

Hors scope

  • Refonte UX des modales
  • Renommage API / back (/admin s’il existe)
  • screens/administrateurs/ (écran création gestionnaire — à évaluer séparément)

Critères d'acceptation

  • Plus de préfixe Admin sur les composants partagés admin+gestionnaire
  • Build Flutter OK
  • Aucune régression fonctionnelle

Milestone

0.1.0 (cleanup — idéalement en fin de jalon)

## Contexte Le panneau « Gestion des utilisateurs » est partagé **admin + gestionnaire**, mais une grande partie du code front est préfixée / rangée `Admin*` : - dossier `frontend/lib/widgets/admin/` - widgets : `AdminChildDetailModal`, `AdminAmEditModal`, `AdminParentEditModal`, `AdminUserCard`, `AdminSelectAmModal`, … - parfois `dashboard_admin.dart` Le préfixe laisse croire que c’est réservé au rôle **administrateur**, alors que c’est du **dashboard staff** (admin *et* gestionnaire). ## Est-ce que ça a du sens ? **Oui**, pour la lisibilité et onboarding dev — mais c’est un **rename mécanique large** (imports partout), donc : - **À faire** quand la surface UI dashboard est un peu stabilisée (après vagues #132 / #153 / fiches), pour limiter les conflits de merge - **Pas bloquant** fonctionnellement pour la 0.1.0 - Peut rester en 0.1.0 en fin de jalon, ou glisser en 0.2.0 si trop de churn ## Proposition de naming À trancher au kick-off du ticket : | Option | Exemple | |--------|---------| | A — préfixe `Staff` | `StaffChildDetailModal` | | B — préfixe `Dashboard` / `Gestion` | `DashboardChildDetailModal` | | C — dossier neutre + noms sans préfixe | `widgets/dashboard/child_detail_modal.dart` → `ChildDetailModal` | Reco : **C** (ou B) — évite un nouveau préfixe ambigu ; garder `AdminManagementWidget` seulement pour l’onglet **Administrateurs**. ## Périmètre - [ ] Inventaire fichiers / classes `Admin*` du dashboard partagé - [ ] Décision naming - [ ] Rename + update imports - [ ] Vérifier routes / tests / docs qui citent les anciens chemins - [ ] Pas de changement comportemental ## Hors scope - Refonte UX des modales - Renommage API / back (`/admin` s’il existe) - `screens/administrateurs/` (écran création gestionnaire — à évaluer séparément) ## Critères d'acceptation - [ ] Plus de préfixe `Admin` sur les composants **partagés** admin+gestionnaire - [ ] Build Flutter OK - [ ] Aucune régression fonctionnelle ## Milestone **0.1.0** (cleanup — idéalement en fin de jalon)
jmartin added this to the 0.1.0 milestone 2026-07-17 15:55:33 +00:00
jmartin added the frontendcleanupgestionnaireadminphase-1v0.1.0 labels 2026-07-17 15:55:33 +00:00
jmartin commented 2026-09-10 21:13:54 +00:00
Author
Owner
Copy Link
Copy Source

Livré sur feature/155-rename-admin-prefix-dashboard

Décision option C appliquée.

  • Widgets partagés Admin* → widgets/dashboard/ sans préfixe (ChildDetailModal, AmEditModal, UserCard, …)
  • Conservé : AdminManagementWidget, screens/administrateurs/*
  • Mini-spec : docs/tmp/155-mini-spec-rename-admin-prefix.md

Pas de changement comportemental — à recetter (dashboard admin + gestionnaire) puis merge.

## Livré sur `feature/155-rename-admin-prefix-dashboard` Décision **option C** appliquée. - Widgets partagés `Admin*` → `widgets/dashboard/` sans préfixe (`ChildDetailModal`, `AmEditModal`, `UserCard`, …) - Conservé : `AdminManagementWidget`, `screens/administrateurs/*` - Mini-spec : `docs/tmp/155-mini-spec-rename-admin-prefix.md` Pas de changement comportemental — à recetter (dashboard admin + gestionnaire) puis merge.
jmartin commented 2026-09-14 08:53:08 +00:00
Author
Owner
Copy Link
Copy Source

Phase 2 #155 — move panels (spécifié)

On reste sur ce ticket pour finir l’option C (pas de #162).

Déjà fait (phase 1)

Widgets partagés Admin* → widgets/dashboard/ sans préfixe (01fb514b sur feature/155-rename-admin-prefix-dashboard).

À faire (phase 2)

Déplacer les panels encore sous widgets/admin/ → widgets/dashboard/.

  • AdminManagementWidget reste dans widgets/admin/ (seul fichier attendu à la fin)
  • Rename mécanique, zéro UX
  • Typo parent_managmant_widget : bonus optionnel

Mini-spec phase 2

docs/tmp/155-suite-move-admin-panels-to-dashboard.md

(Inventaire, arborescence cible, plan, grep, hors scope.)

## Phase 2 #155 — move panels (spécifié) On **reste sur ce ticket** pour finir l’option C (pas de #162). ### Déjà fait (phase 1) Widgets partagés `Admin*` → `widgets/dashboard/` sans préfixe (`01fb514b` sur `feature/155-rename-admin-prefix-dashboard`). ### À faire (phase 2) Déplacer les **panels** encore sous `widgets/admin/` → `widgets/dashboard/`. - `AdminManagementWidget` **reste** dans `widgets/admin/` (seul fichier attendu à la fin) - Rename mécanique, **zéro** UX - Typo `parent_managmant_widget` : bonus optionnel ### Mini-spec phase 2 `docs/tmp/155-suite-move-admin-panels-to-dashboard.md` (Inventaire, arborescence cible, plan, grep, hors scope.)
jmartin commented 2026-09-14 10:46:51 +00:00
Author
Owner
Copy Link
Copy Source

Ferm� � merg� dans `develop`

  • Merge : merge(#155) � option C compl�te (phase 1 + phase 2)
  • Phase 1 : widgets partag�s sans pr�fixe Admin ? `widgets/dashboard/`
  • Phase 2 : panels/wizards/validation/common ? `widgets/dashboard/` ; `widgets/admin/` ne conserve que `AdminManagementWidget`
  • Recette OK (dashboard admin + gestionnaire / onglet Dossiers)
## Ferm� � merg� dans \`develop\` - Merge : merge(#155) � option C compl�te (phase 1 + phase 2) - Phase 1 : widgets partag�s sans pr�fixe Admin ? \`widgets/dashboard/\` - Phase 2 : panels/wizards/validation/common ? \`widgets/dashboard/\` ; \`widgets/admin/\` ne conserve que \`AdminManagementWidget\` - Recette OK (dashboard admin + gestionnaire / onglet Dossiers)
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
a11y
Accessibilité
 admin
Administration
 api
auth
Authentification
 backend
Backend NestJS
 bug
Something isn't working
 cdc
Cahier des charges
 cleanup
Nettoyage/Refactoring
 critique
Critique
 database
Base de données PostgreSQL
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 email
Email/Notifications
 enhancement
New feature or request
 frontend
Frontend Flutter/Web
 gestionnaire
Gestionnaire
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 infra
Infrastructure Docker/CI-CD
 invalid
This doesn't seem right
 juridique
Juridique/Légal
 monitoring
Monitoring/Logs
 on-premise
Configuration on-premise
 p0-bloquant
Priorité 0 - BLOQUANT
 p1-bloquant
Priorité 1 - BLOQUANT
 p2
Priorité 2
 p3
Priorité 3
 p4
Priorité 4
 phase-1
Phase 1
 question
Further information is requested
 rgpd
RGPD/Conformité
 security
Sécurité
 tests
Tests unitaires/E2E
 ui
upload
Upload fichiers
 ux
UX/UI
 v0.1.0
Issue rattachée au périmètre release 0.1.0
 wontfix
This will not be worked on
No labels admin cleanup frontend gestionnaire phase-1 v0.1.0
Milestone
No items
No Milestone 0.1.0
Projects
Clear projects
No projects
Assignees
Clear assignees
nnasarre jmartin (MARTIN Julien)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jmartin/petitspas#155