Fix #968 : Mentionne le type d'affaire dans les vues concernant les devis/factures

CroissanceCommune / autonomie

Autonomie est un progiciel de gestion à l'usage des CAE

GNU General Public License v3.0

20 stars 12 forks source link

Closed JocelynDelalande closed 5 years ago

JocelynDelalande commented 5 years ago

Reste (entre autres):

[x] Retirer code mort https://github.com/CroissanceCommune/autonomie/blob/master/autonomie/forms/tasks/base.py#L422-L454
[x] Déplacer business_type_label dans un dossier mieux approprié
[x] Afficher le type d'affaire sur /estimations/:id et (lecture seule ? lecture/écriture)
[x] Idem sur /invoices/:id
[x] ajouter validateurs pour la sélection de type d'affaire devis/facture
[x] suggestion de @SophieWeb
[x] Build JS

JocelynDelalande commented 5 years ago

Question review :

la validation du sélecteur de type d'affaire est faite, mais déclenche une 500 au lieu d'une 400. Y-a-t-il des exemples d'endroits où c'est fait correctement dans un cas similaire (backbone+appel API). En même temps, pas dramatique, on ne peut tomber dans cette erreur de validation que si l'on trafique le DOM.

tonthon commented 5 years ago

Il y a des améliorations à apporter à la gestion des retours d'erreur dans les pages JS. Je n'ai pas encore de solutions pour cela.