-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modification intitulés pour mieux distinguer réglementations temporaires et permanentes #1069
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1069 +/- ##
=========================================
Coverage 98.79% 98.79%
Complexity 1745 1745
=========================================
Files 351 351
Lines 7580 7580
=========================================
Hits 7489 7489
Misses 91 91 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
<source>regulation.general_info.description</source> | ||
<target>Description</target> | ||
<target>Intitulé</target> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça introduit une différence de nommage entre le code ("description") et le langage oral qu'on utilisera ("intitulé"), ce qui n'est pas bon à moyen-long terme (principe du DDD : ubiquitous language = 1 même langage utilisé par tout le monde, + le code doit refléter la terminologie métier)
Peut-être qu'il faudrait en profiter pour renommer le champ en title
, mais dans ce cas-là le faire dans une PR séparée ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok pour faire ça dans une PR séparée ! Donc il faudrait même changer en base de données le nom du champ dans la table c'est bien ce que tu voulais dire @florimondmanca ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oui dans l'idéal... cc @mmarchois
Ça peut aussi te faire un exercice de refacto :) (renommer un champ ça arrive)
Cette PR traite simplement les changements d'intitulé