💫 Change verbose name and traduction#4008
💫 Change verbose name and traduction#4008TheoLechemia wants to merge 1 commit intoGeotrekCE:masterfrom
Conversation
Django override traduction between apps. We need a different trad for label in tourism app
Passing run #8432 ↗︎
Details:
Review all test suite changes for PR #4008 ↗︎ |
|||||||||||||||
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4008 +/- ##
=======================================
Coverage 98.37% 98.37%
=======================================
Files 296 296
Lines 22560 22560
=======================================
Hits 22193 22193
Misses 367 367 ☔ View full report in Codecov by Sentry. |
|
|
||
| class TouristicEventOrganizer(TimeStampedModelMixin): | ||
| label = models.CharField(verbose_name=_("Label"), max_length=256) | ||
| label = models.CharField(verbose_name=_("OrganizerLabel"), max_length=256) |
There was a problem hiding this comment.
Je pense qu'on s'est pas compris la dernière fois. SI il n'y a pas de pb de compatibilité (api v2, parsers etc), le mieux est encore de renommer le champ directement plutot que d'introduire des incohérences dans les trads (toutes les trads correspondent aux champs). Là, le champ s'apelle label, mais en réalité, 'name' serait plus approrié, et donc Name en EN et Nom en français (étant donné que c'est une personne physique ou morale)
There was a problem hiding this comment.
Effectivement, j'avais pas compris ça.
Je corrige
There was a problem hiding this comment.
Oui name serait mieux que label. Mais donc il faut aussi renommer le champs dans la BDD
There was a problem hiding this comment.
Oui et potentiellement dans les parsers etc etc..
Je me demande s'il ne faudra pas résoudre ce problème de mauvaise surcouche des trads entre apps en fait
There was a problem hiding this comment.
Oui et potentiellement dans les parsers etc etc.. Je me demande s'il ne faudra pas résoudre ce problème de mauvaise surcouche des trads entre apps en fait
Je pense que c'est plus lié à django qu'a geotrek-admin. Dans les dernieres versions, ils ont amélioré pas mal de chose à ce niveau. Par exemple, lors d'un makemessages, ils ne met plus à jour la date du fichier si aucune trad n'a été modifiée. Je jetterai un oeil sur ma branche django 4.2 voir si çà y a du mieux à ce niveau
3 participants
No description provided.