Skip to content

Commit

Permalink
Remove required from tech_name mixin
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
guewen authored and lmignon committed Oct 19, 2022
1 parent b63ea4b commit 3b7ee2a
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions server_environment/models/server_env_tech_name_mixin.py
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down

0 comments on commit 3b7ee2a

Please sign in to comment.