Refonte Trésorie > Facture - Ajout / Edition#2185
Conversation
vgreb
force-pushed
the
refacto/accounting-invoice-edit
branch
4 times, most recently
from
d755723 to
d8a8313
Compare
|
@Mopolo @stakovicz @agallou En partant du principe qu'une facture est générée à partir d'un devis, je n'autorise pas l'ajout ou la suppression d'une ligne de détails. Est-ce que ça raccord avec les process de la trésorerie ? |
vgreb
force-pushed
the
refacto/accounting-invoice-edit
branch
from
d8a8313 to
bc7e3af
Compare
Je pense que oui. Ça pourra faire l'objet d'une évolution plus tard sinon. |
| path: /invoices/list | ||
| defaults: {_controller: AppBundle\Controller\Admin\Accounting\Invoice\ListInvoiceAction} | ||
|
|
||
| admin_accounting_invoices_edit: |
There was a problem hiding this comment.
Il manque l'ajout de cette route dans le menu pour avoir la navigation ouverte à gauche.
There was a problem hiding this comment.
suggestion : Je sais que cela provient fortement d'un devis, mais je suggère de supprimer au maximum les options inutiles pour alléger le code :
'required' => true,: comme c'est la valeur par défautnew Assert\Type('string'),: préférer le typage au niveau de la propriété pour que ce soit plus robuste'empty_data' => '',: comme c'est la valeur par défaut
C'est peut être hors scope à voir dans un autre ticket comme on a la même chose sur le QuotationType
2 participants
resolves #2007