-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fléchage du département à la place du simulateur sur la home (GUH - V1.3) #435
Conversation
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.
J'ai laissé une ou deux questions, sinon Nickel !
envergo/moulinette/models.py
Outdated
@@ -709,6 +710,39 @@ def __str__(self): | |||
return self.department.get_department_display() | |||
|
|||
|
|||
class HaieDepartmentConfig(models.Model): |
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.
Question : est-ce que tu as considéré le fait d'avoir un objet commun dont hériteraient les objets config pour aménagement et haie ?
Ici, on a un objet MoulinetteConfig
qui ne sert que pour aménagement tandis que HaieDepartementConfig
sert pour haie. J'ai l'impression qu'on manque un peu de cohérence dans le nommage des classes, qu'en penses-tu ?
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.
(pour l'historique : vu en discussion privée)
Ce tikcet