Make some fields optional

SDESK-7441
superdesk · Nov 29, 2024 · d6cb71c · d6cb71c
1 parent b85a4e2
commit d6cb71c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/server/planning/types/common.py b/server/planning/types/common.py
@@ -237,7 +237,7 @@ class ScheduledUpdate:
 @dataclass
 class PlanningCoverage:
     # Identifiers
-    coverage_id: fields.Keyword
+    coverage_id: fields.Keyword | None = None
     original_coverage_id: fields.Keyword | None = None
     guid: fields.Keyword | None = None
 

diff --git a/server/planning/types/event.py b/server/planning/types/event.py
@@ -193,7 +193,9 @@ class EventResourceModel(BasePlanningModel, LockFieldsMixin):
     participant: list[KeywordQCodeName | None] = Field(default_factory=list)
     participant_requirement: list[KeywordQCodeName | None] = Field(default_factory=list)
     organizer: list[KeywordQCodeName | None] = Field(default_factory=list)
-    event_contact_info: Annotated[list[fields.ObjectId], validate_data_relation_async("contacts")]
+    event_contact_info: Annotated[list[fields.ObjectId], validate_data_relation_async("contacts")] = Field(
+        default_factory=list
+    )
     language: fields.Keyword | None = None
     languages: list[fields.Keyword] = Field(default_factory=list)
 
@@ -259,7 +261,7 @@ class EventResourceModel(BasePlanningModel, LockFieldsMixin):
     # HACK: end. We'll try to move these hacks somewhere else
 
     extra: Annotated[dict[str, Any], fields.dynamic_mapping()] = Field(default_factory=dict)
-    translations: Annotated[list[Translation], fields.nested_list()]
+    translations: Annotated[list[Translation], fields.nested_list()] = Field(default_factory=list)
 
     # This is used from the EmbeddedCoverage form in the Event editor
     # This list is NOT stored with the Event