Je commence à relire.
(Même question ici sur la pertinence de chopper les motifs d’annulation par leur slug plutôt que par leur libellé ?)
Ok mais à ne pas taper directement un .get_or_create sur le libellé on va se retrouver avec des motifs d’annulation dont le slug est en anglais, et d’autres, définies par les régisseurs, dont le slug est en français. Est-ce que c’est grave ?
Ça m’a l’air un peu violent d’avoir un TextField juste pour ce qui a priori est une description succincte, non ?
(Je vais refaire un tour à tête reposée mais a priori on n’est pas loin du ack.)
Ok j’avais clairement pas saisi cette nuance dans les traductions, très bien.
Ah ok j’avais loupé ça, merci.
Pas compris ici pourquoi il y a besoin d’un contexte i.e. de quelles autres façons ce “Edit pricing” pourrait se traduire ?
Par principe je me disais peut-être mettre une valeur par défaut field='pricing' pour ce nouveau paramètre histoire de ne pas casser l’interface existante (?)
Un premier bout de relecture, surtout des remarques de forme, dis-moi ce que tu en penses, je referai un tour ensuite.
Un if pas nécessaire ici, il reprend exactement la même condition que le if du bloc parent.
Question d’UI, à relire l’héritage de gabarits j’ai l’impression qu’on ajoute une barre latérale vide à chaque fois, ici et dans les gabarits suivants, je loupe un truc ou bien c’est volontaire que d’avoir hérité de manager_pricing_parameters pour ensuite effacer le contenu de la barre latérale qui y était défini ?
(Pareil ici peut-être juste un elif car les conditions sont mutuellement exclues ?)