From 6ca6d36ac1c7b4337ac80d1ab9111451a60112e4 Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Thu, 14 Jan 2021 09:12:48 +0100 Subject: [PATCH] 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 fcf676c..3bdafa4 100644 --- a/server_environment/models/server_env_tech_name_mixin.py +++ b/server_environment/models/server_env_tech_name_mixin.py @@ -34,8 +34,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"