Fixed issue in Admin model to allow the 'email' field to be either nu…

…ll or excluded
Azure · Oct 10, 2023 · b37ea4f · b37ea4f
1 parent 2571910
commit b37ea4f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/engine/app/models.py b/engine/app/models.py
@@ -435,13 +435,13 @@ class Admin(BaseModel):
     name: str
     email: Optional[EmailStr] = None
     id: UUID
-        
+
     @model_validator(mode='before')
     @classmethod
     def check_email(cls, data: Any) -> Any:
         if isinstance(data, dict):
             if 'type' in data and 'email' in data:
-                if data['type'] == "Principal":
+                if data['type'] == "Principal" and data['email'] is not None:
                     raise ValueError("email should not be set for 'principal' type")
             if 'type' in data and 'email' not in data:
                 if data['type'] == "User":