betagouv / complements-alimentaires

MIT License
5 stars 0 forks source link

Modifs de l'admin pour faciliter les ajouts de nouveaux ingrédients #1304

Open pletelli opened 2 days ago

pletelli commented 2 days ago

TODO : fix du calcul de l'article au save_model()

pletelli commented 1 day ago

J'ai pas de contexte alors p-e pas la meilleure personne de faire la revue mais ça me paraît raisonnable. Petit point : je crois que l'ajout d'une valeur qui vient d'un autre objet (standard_name) comme __str__ va rajouter les warnings de query N + 1, mais je suis pas certaine.

Hmmm c'est possible. Je ne maîtrise pas très bien les causes des N +1. Mais en tout cas, je voulais plutôt renvoyer le name que le standard_name : 2bbd1db91c68a94421ad2ebf72e1c36ce16779a8 Merci !

Pour le contexte : c'est juste pour visualiser plus facilement les objets que je manipule éventuellement dans le shell Django