[IMP] Labels : font standardization, Organic info standardization

[quentinDupont]

Changement pour clarification réglementaires bio

[legalsylvain]

Hello !
je pense que les champs Groupe de pays / Pays / Région / Département devraient être suffixé par "des MP" (ou "des matières premières").
Sans ça, ce n'est pas explicite :

• dans une édition en lot
• dans les vues listes
• quand on fait une recherche.

Qu'en penses-tu ?

[quentinDupont]

Hello ! je pense que les champs Groupe de pays / Pays / Région / Département devraient être suffixé par "des MP" (ou "des matières premières"). Sans ça, ce n'est pas explicite :

* dans une édition en lot

* dans les vues listes

* quand on fait une recherche.

Qu'en penses-tu ?

yes jsuis d'ac !
j'avais peur que ça soit lourd, mais si j'écris "MP" ça va :)
je change aussi les termes pour les vues inline d'ailleurs, ça m'y a fait pensé merci !

[legalsylvain]

je change aussi les termes pour les vues inline d'ailleurs, ça m'y a fait pensé merci !

Nop !
en fait, si tu renommes le champs en mode
country_id = fields.Many2one(string='Country (Raw Material)')

ça le fait partout. Du coup, pas besoin de changer dans chaque vue inline ou autre.

"Matière premières" c'est un peu long, mais "MP", c'est pas ultra parlant. (surtout pour les épiciers).
"Ingrédients" ?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c3e231d) 78.87% compared to head (342983a) 78.91%.

❗ Current head 342983a differs from pull request most recent head ea80940. Consider uploading reports for the commit ea80940 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             12.0     #344      +/-   ##
==========================================
+ Coverage   78.87%   78.91%   +0.04%     
==========================================
  Files         203      203              
  Lines        2007     2011       +4     
  Branches      351      351              
==========================================
+ Hits         1583     1587       +4     
  Misses        406      406              
  Partials       18       18              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[legalsylvain]

j'y pense, est-ce que ce que tu pourrais rajouter des helpers sur tous les champs origine, pour expliquer comment renseigner ce champ ?
je veux dire, si c'est un gateau au chocolat, quel est l'origine qu'on doit mettre ? (la farine france, le chocolat ethiopie, etc...)

[quentinDupont]

Nop ! en fait, si tu renommes le champs en mode country_id = fields.Many2one(string='Country (Raw Material)')

ça le fait partout. Du coup, pas besoin de changer dans chaque vue inline ou autre.

Oui oui c'est ce que j'ai fait
Je parlais du nom des menus :)

[quentinDupont]

"Matière premières" c'est un peu long, mais "MP", c'est pas ultra parlant. (surtout pour les épiciers).
"Ingrédients" ?

Oui c'est vrai que sur les étiquettes c'est écrit Ingrédients
Mais là ça fait un peu long p-e, mais ouais jvais tenter ça

[legalsylvain]

quick code review.

[legalsylvain]

MeP du 3/12/2023
/ocabot merge patch

[github-grap-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-344-by-legalsylvain-bump-patch, awaiting test results.

[github-grap-bot]

Congratulations, your PR was merged at 24a11dd. Thanks a lot for contributing to grap. ❤️