From 9ff914d3f2bbc01191f27158138ddfe0537cd874 Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Thu, 14 Jan 2021 09:12:48 +0100 Subject: [PATCH 1/2] Remove required from tech_name mixin In some cases, we want to use a tech_name, but we do not use server environment on all the records, so tech_name should not be required. --- server_environment/models/server_env_tech_name_mixin.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/server_environment/models/server_env_tech_name_mixin.py b/server_environment/models/server_env_tech_name_mixin.py index d865184..cb419c2 100644 --- a/server_environment/models/server_env_tech_name_mixin.py +++ b/server_environment/models/server_env_tech_name_mixin.py @@ -38,8 +38,7 @@ class ServerEnvTechNameMixin(models.AbstractModel): # TODO: could leverage the new option for computable / writable fields # and get rid of some onchange / read / write code. tech_name = fields.Char( - required=True, - help="Unique name for technical purposes. " "Eg: server env keys.", + help="Unique name for technical purposes. Eg: server env keys.", ) _server_env_section_name_field = "tech_name" From 4ca6a71777702565d477d51d9a6bd5e2d63244f4 Mon Sep 17 00:00:00 2001 From: Simone Orsi Date: Tue, 10 May 2022 08:16:13 +0200 Subject: [PATCH 2/2] server_environment: do not copy tech_name --- server_environment/models/server_env_tech_name_mixin.py | 1 + 1 file changed, 1 insertion(+) diff --git a/server_environment/models/server_env_tech_name_mixin.py b/server_environment/models/server_env_tech_name_mixin.py index cb419c2..59c9982 100644 --- a/server_environment/models/server_env_tech_name_mixin.py +++ b/server_environment/models/server_env_tech_name_mixin.py @@ -39,6 +39,7 @@ class ServerEnvTechNameMixin(models.AbstractModel): # and get rid of some onchange / read / write code. tech_name = fields.Char( help="Unique name for technical purposes. Eg: server env keys.", + copy=False, ) _server_env_section_name_field = "tech_name"