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

feat(transport) : écran ajout transporteur — layout + formulaire principal (ERP-165) #122

Open
tristan wants to merge 7 commits from feat/erp-165-carrier-new into develop
pull from: feat/erp-165-carrier-new
merge into: MALIO-DEV:develop
Conversation 1 Commits 7 Files Changed 14
+1445 -35
tristan commented 2026-06-16 15:05:46 +00:00
Owner
Copy Link
Copy Source

Layout onglets + POST /api/carriers. Ticket ERP-165.

⚠️ Stack sur #121 (ERP-164, branche feat/erp-164-carriers-list, non encore mergée) : à merger APRÈS #121. Tant que #121 n'est pas mergé, ce diff inclut ses commits.

Périmètre : page /carriers/new (layout + barre d'onglets Qualimat/Adresses/Contacts/Prix à validation incrémentale), formulaire principal (Nom / Certification / Affréter) → POST /api/carriers, composable useCarrierForm (POST principal + PATCH partiels par onglet). Champs conditionnels + saisie assistée QUALIMAT au ticket suivant (ERP-166).

Layout onglets + POST /api/carriers. Ticket ERP-165. ⚠️ Stack sur #121 (ERP-164, branche `feat/erp-164-carriers-list`, non encore mergée) : à merger APRÈS #121. Tant que #121 n'est pas mergé, ce diff inclut ses commits. Périmètre : page /carriers/new (layout + barre d'onglets Qualimat/Adresses/Contacts/Prix à validation incrémentale), formulaire principal (Nom / Certification / Affréter) → POST /api/carriers, composable useCarrierForm (POST principal + PATCH partiels par onglet). Champs conditionnels + saisie assistée QUALIMAT au ticket suivant (ERP-166).
tristan added 4 commits 2026-06-16 15:05:47 +00:00
feat(transport) : page répertoire transporteurs (ERP-164)
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 2m54s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m33s
Details
1ef4215ebf
feat(transport) : filtres checkbox, toggle « Voir les archivés », transporteurs dans Administration (ERP-164)
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 3m9s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m36s
Details
8046de76c6
chore(frontend) : bump @malio/layer-ui ^1.7.12 + commentaire useSuppliersRepository
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 3m14s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m31s
Details
597c63bb2e
feat(transport) : écran ajout transporteur — layout + formulaire principal (ERP-165)
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 3m11s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Has been cancelled
Details
5734aaef54
tristan added the frontM4-Transporteurtype/feat labels 2026-06-16 15:05:56 +00:00
tristan added 1 commit 2026-06-16 15:09:52 +00:00
fix(transport) : centre verticalement la case « Affréter » sur la ligne de champ (ERP-165)
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 3m2s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m24s
Details
f1b18cfbbe
tristan added 2 commits 2026-06-17 06:40:04 +00:00
Merge remote-tracking branch 'origin/develop' into feat/erp-164-carriers-list
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 3m1s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m20s
Details
45158af920
Merge branch 'feat/erp-164-carriers-list' into feat/erp-165-carrier-new
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 2m54s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m22s
Details
4202977950
tristan commented 2026-06-17 13:57:55 +00:00
Author
Owner
Copy Link
Copy Source

Review front — ERP-165 (écran ajout, formulaire principal)

Verdict : mergeable (1 réserve UX à confirmer).

Fidèle à clients/new.vue : erreurs 422 inline via useFormErrors, appels { toast: false }, bouton Valider toujours actif, payload qui omet les champs vides pour laisser la 422 métier porter sur le bon champ. Tests pertinents (pré-check nom vide, payload exact, headers ld+json, 409, 422 mappé, gating onglets).

Findings

  • MINOR — Sur un 409 doublon (name), le code fait setError('name', …) ET toast.error(…) → doublon visuel (l'erreur est déjà affichée inline sous le champ). C'est toléré par la référence useCategoryForm (RG-1.07), donc pas une violation stricte, mais à trancher côté UX : l'inline seul suffirait.
  • NIT — Plusieurs sorties du composable (patchCarrier, editMode, validated, buildMainPayload) ne sont pas encore consommées (placeholders des tickets suivants) — assumé, couvert par les tests, pas de code mort réel.

Incrément reviewé : erp-164..erp-165 (front).

## Review front — ERP-165 (écran ajout, formulaire principal) **Verdict : mergeable** (1 réserve UX à confirmer). Fidèle à `clients/new.vue` : erreurs 422 inline via `useFormErrors`, appels `{ toast: false }`, bouton Valider toujours actif, payload qui omet les champs vides pour laisser la 422 métier porter sur le bon champ. Tests pertinents (pré-check nom vide, payload exact, headers `ld+json`, 409, 422 mappé, gating onglets). **Findings** - **MINOR** — Sur un **409 doublon** (`name`), le code fait `setError('name', …)` **ET** `toast.error(…)` → doublon visuel (l'erreur est déjà affichée inline sous le champ). C'est toléré par la référence `useCategoryForm` (RG-1.07), donc pas une violation stricte, mais à trancher côté UX : l'inline seul suffirait. - **NIT** — Plusieurs sorties du composable (`patchCarrier`, `editMode`, `validated`, `buildMainPayload`) ne sont pas encore consommées (placeholders des tickets suivants) — assumé, couvert par les tests, pas de code mort réel. > Incrément reviewé : `erp-164..erp-165` (front).
Some checks are pending
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 2m54s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m22s
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/erp-165-carrier-new:feat/erp-165-carrier-new
git checkout feat/erp-165-carrier-new
Sign in to join this conversation.
Reviewers
No Reviewers
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 M4-Transporteur 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#122
Delete Branch "feat/erp-165-carrier-new"

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?