Feat/employee receives bdc with or without lic - #928
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
- Keep the "Envoyer" button visible regardless of send status - Guard submit against empty inputs and surface a field-level error - Drop the now-unused required/disabled validation logic
- Add the "remis au format papier" checkbox directly inside ControlSendEmailNoLicModal, driven by handDelivered/handleHandDeliveredChange - Disable the driver email field based on handDelivered instead of a derived disableDriverInput prop - Only show the "Remis au conducteur (via Mobilic)" badge in LIC context, since it reflects availability in the driver's Mobilic account
|
gaspard-lonchampt
left a comment
gaspard-lonchampt
left a comment
There was a problem hiding this comment.
🟠 Défaut forAdmin inversé dans le hook (useControlBulletinActions.js)
Le défaut est false (conducteur), alors que le back-end a for_admin=True (gestionnaire) et que le comportement historique était « gestionnaire ». Tous les appelants actuels passent forAdmin explicitement, donc pas de bug actif, mais un futur appel handleSend(emails) sans le flag enverrait silencieusement au conducteur. Je suggère dʼaligner le défaut sur true (ou de rendre le paramètre obligatoire) pour rester cohérent avec le back.
🟠 À confirmer : double pastille « Remis au conducteur » (ControllerControlBottomMenu.js)
Sur un contrôle LIC, la pastille « Remis au conducteur (via Mobilic) » (condition isBulletinAvailableForDriver && !isNoLicContext) peut coexister avec la nouvelle « Remis au conducteur (papier/via email) » (condition sentToDriver || handDelivered). Deux pastilles « Remis au conducteur » côte à côte : voulu (deux canaux distincts) ou redondant ?
3 participants
https://trello.com/c/XZdtib1C