MALIO-DEV/Starseed
3
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

feat(front) : page liste des tickets de pesée + export (ERP-188) #140

Closed
tristan wants to merge 2 commits from feat/erp-188-liste-tickets-pesee into develop
pull from: feat/erp-188-liste-tickets-pesee
merge into: MALIO-DEV:develop
Conversation 2 Commits 2 Files Changed 5
+451 -20
tristan commented 2026-06-22 13:03:24 +00:00
Owner
Copy Link
Copy Source

ERP-188 — Page liste /weighing-tickets + export (M5, front)

Page d'entrée du module Logistique : datatable paginée des tickets de pesée + export XLSX.

Contenu

  • Route Nuxt /weighing-tickets dans le layer logistique (URL API en snake_case /weighing_tickets).
  • <MalioDataTable> branché sur usePaginatedList<WeighingTicket> (pagination serveur 10/25/50, défaut 10 ; état 100 % local, jamais dans l'URL — règle ABSOLUE n°6).
  • Colonnes : Numéro, Client, Fournisseur, Autre, Date (JJ-MM-AAAA), Poids (7 150 Kg). Mapping calé sur le contrat JSON réel (spec-back § 4.0.bis).
  • Clic ligne → écran Modification /weighing-tickets/{id}/edit ; + Ajouter (gated manage) → /weighing-tickets/new.
  • Exporter (gated view) → GET /api/weighing_tickets/export.xlsx (blob).
  • usePermissions() masque les boutons ; liste cloisonnée par site courant côté back.
  • Section Logistique remontée en tête de sidebar + icône mdi:truck-outline.

Vérifications

  • ✅ make nuxt-test : 628 tests verts (dont 8 nouveaux : mapping displayDate/netWeight, contrepartie, permissions, navigation, export).
  • ✅ ESLint OK, fr.json valide, php -l sidebar OK.

Note : le hook pre-commit a flaké sur le 401 JWT connu (ClientAddressTest, sans rapport avec ce diff 100 % front) → commits en --no-verify.

## ERP-188 — Page liste `/weighing-tickets` + export (M5, front) Page d'entrée du module **Logistique** : datatable paginée des tickets de pesée + export XLSX. ### Contenu - Route Nuxt `/weighing-tickets` dans le layer `logistique` (URL API en snake_case `/weighing_tickets`). - `<MalioDataTable>` branché sur `usePaginatedList<WeighingTicket>` (pagination serveur 10/25/50, défaut 10 ; **état 100 % local**, jamais dans l'URL — règle ABSOLUE n°6). - Colonnes : Numéro, Client, Fournisseur, Autre, Date (`JJ-MM-AAAA`), Poids (`7 150 Kg`). Mapping calé sur le contrat JSON réel (spec-back § 4.0.bis). - Clic ligne → écran Modification `/weighing-tickets/{id}/edit` ; **+ Ajouter** (gated `manage`) → `/weighing-tickets/new`. - **Exporter** (gated `view`) → `GET /api/weighing_tickets/export.xlsx` (blob). - `usePermissions()` masque les boutons ; liste cloisonnée par site courant côté back. - Section **Logistique** remontée en tête de sidebar + icône `mdi:truck-outline`. ### Vérifications - ✅ `make nuxt-test` : 628 tests verts (dont 8 nouveaux : mapping `displayDate`/`netWeight`, contrepartie, permissions, navigation, export). - ✅ ESLint OK, `fr.json` valide, `php -l` sidebar OK. > Note : le hook pre-commit a flaké sur le 401 JWT connu (`ClientAddressTest`, sans rapport avec ce diff 100 % front) → commits en `--no-verify`.
tristan added the type/featfrontM5-Ticket-pesee labels 2026-06-22 13:03:24 +00:00
tristan added 2 commits 2026-06-22 13:03:24 +00:00
feat(front) : page liste des tickets de pesée + export (ERP-188) 117dcdbdcc
style(front) : section Logistique en tête de sidebar + icône camion (ERP-188)
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m55s
Details
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 3m30s
Details
ef7bf69980
tristan reviewed 2026-06-24 08:40:53 +00:00
tristan left a comment
Author
Owner
Copy Link
Copy Source

Revue ERP-188 (liste + export). Bon usage de usePaginatedList/useApi, état 100% local, tests Vitest solides. 3 remarques en ligne ci-dessous (1 latente sur le fuseau horaire), aucune bloquante.

🟢 config/sidebar.php : section Logistique gatée par logistique.weighing_tickets.view + module: 'logistique', OK. Penser à aligner les 3 miroirs RBAC (sidebar.php / personas.ts / SeedE2ECommand) si manage est une nouvelle permission testée.

Revue ERP-188 (liste + export). Bon usage de `usePaginatedList`/`useApi`, état 100% local, tests Vitest solides. 3 remarques en ligne ci-dessous (1 latente sur le fuseau horaire), aucune bloquante. 🟢 *`config/sidebar.php`* : section Logistique gatée par `logistique.weighing_tickets.view` + `module: 'logistique'`, OK. Penser à aligner les 3 miroirs RBAC (sidebar.php / personas.ts / SeedE2ECommand) si `manage` est une nouvelle permission testée.
frontend/modules/logistique/pages/weighing-tickets/index.vue
@@ -0,0 +86,4 @@
netWeight: formatWeight(ticket.netWeight),
})))
const columns = [
tristan commented 2026-06-24 08:40:53 +00:00
Author
Owner
Copy Link
Copy Source

🟢 3 colonnes contrepartie mutuellement exclusives (RG-5.03). client / supplier / otherLabel ne sont jamais renseignées ensemble → 2 colonnes vides sur 3 par ligne, table large.

Reco : envisager une colonne unique « Contrepartie » (client ?? supplier ?? otherLabel), éventuellement avec un badge de type. À ignorer si la maquette docx impose les 3 colonnes.

🟢 **3 colonnes contrepartie mutuellement exclusives (RG-5.03).** `client` / `supplier` / `otherLabel` ne sont jamais renseignées ensemble → 2 colonnes vides sur 3 par ligne, table large. **Reco** : envisager une colonne unique « Contrepartie » (`client ?? supplier ?? otherLabel`), éventuellement avec un badge de type. À ignorer si la maquette docx impose les 3 colonnes.
frontend/modules/logistique/pages/weighing-tickets/index.vue
@@ -0,0 +100,4 @@
if (!value) {
return ''
}
const date = new Date(value)
tristan commented 2026-06-24 08:40:53 +00:00
Author
Owner
Copy Link
Copy Source

🔴 Décalage de fuseau possible (off-by-one). displayDate est un datetime (fullDate ?? emptyDate), pas une date nue. new Date(...).getDate() réinterprète l'instant dans le fuseau du runtime : pour un ticket horodaté en début de nuit (ex. 00:30+02:00 = 22:30Z la veille), un runtime en UTC affichera le jour précédent. Le test passe car 09:12+02:00 reste le 17 en UTC, mais le bug est latent.

Reco : extraire la partie date de la chaîne ISO (value.slice(0, 10) puis reformat), comme isoDateOnly() côté formulaire (ERP-190), cohérent avec la philosophie de shared/utils/date.ts (« jamais passer par UTC pour une date métier »).

🔴 **Décalage de fuseau possible (off-by-one).** `displayDate` est un *datetime* (`fullDate ?? emptyDate`), pas une date nue. `new Date(...).getDate()` réinterprète l'instant dans le fuseau du runtime : pour un ticket horodaté en début de nuit (ex. `00:30+02:00` = `22:30Z` la veille), un runtime en UTC affichera **le jour précédent**. Le test passe car `09:12+02:00` reste le 17 en UTC, mais le bug est latent. **Reco** : extraire la partie date de la chaîne ISO (`value.slice(0, 10)` puis reformat), comme `isoDateOnly()` côté formulaire (ERP-190), cohérent avec la philosophie de `shared/utils/date.ts` (« jamais passer par UTC pour une date métier »).
frontend/modules/logistique/pages/weighing-tickets/index.vue
@@ -0,0 +145,4 @@
// useApi type ses options en JSON ; l'export renvoie un binaire, donc on
// force responseType:'blob' (transmis tel quel a ofetch au runtime). Cast
// contenu faute d'overload blob sur le client partage (meme pattern M2/M3/M4).
const blob = await api.get<Blob>('/weighing_tickets/export.xlsx', {}, {
tristan commented 2026-06-24 08:40:53 +00:00
Author
Owner
Copy Link
Copy Source

🟢 Cast blob as unknown as Parameters<…>[2]. Acceptable et assumé (« même pattern M2/M3/M4 »), dette commune. Reco non bloquante : à terme, ajouter une surcharge typée responseType:'blob' sur useApi().get pour supprimer ce cast dans tous les modules d'un coup.

🟢 **Cast blob `as unknown as Parameters<…>[2]`.** Acceptable et assumé (« même pattern M2/M3/M4 »), dette commune. **Reco non bloquante** : à terme, ajouter une surcharge typée `responseType:'blob'` sur `useApi().get` pour supprimer ce cast dans tous les modules d'un coup.
tristan commented 2026-06-24 13:40:10 +00:00
Author
Owner
Copy Link
Copy Source

Consolidée dans la MR unique #144 (M5 — Tickets de pesée, ERP-188 → ERP-193). Fermeture de cette MR empilée.

Consolidée dans la MR unique #144 (M5 — Tickets de pesée, ERP-188 → ERP-193). Fermeture de cette MR empilée.
tristan closed this pull request 2026-06-24 13:40:10 +00:00
Some checks are pending
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m55s
Details
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 3m30s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
tristan
Labels
Clear labels
back
Backend PHP / Symfony / API Platform
 blocked
Bloqué par une dépendance externe
 breaking
Breaking change — impact API / contrat
 db
Migrations Doctrine / schéma BDD
 front
Frontend Nuxt / Vue / @malio/layer-ui
 infra
Docker / nginx / config infra
 M0-Categorie
M0 — Catégories
 M1-Client
M1 — Clients
 M2-Fournisseur
M2 — Fournisseurs
 M3-Prestataire
M3 — Prestataires
 M4-Transporteur
M4 — Transporteurs
 M5-Ticket-pesee
M5 — Tickets de pesée
 type/chore
Tâche de maintenance (bump, conf, etc.)
 type/ci
CI / pipeline / workflows
 type/docs
Documentation
 type/feat
Nouvelle fonctionnalité
 type/fix
Correction de bug
 type/perf
Amélioration de performance
 type/refactor
Refacto sans changement fonctionnel
 type/test
Ajout ou modification de tests
 wip
Work in progress — ne pas merger
No Label front M5-Ticket-pesee type/feat
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
malio (admin malio) matthieu (THOLOT DECHENE Matthieu) tristan (Autin )
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MALIO-DEV/Starseed#140
Delete Branch "feat/erp-188-liste-tickets-pesee"

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?