Merge pull request #188 from Azure/admin-users-fix

Admin Users Fix
Azure · Oct 10, 2023 · 71c7d09 · 71c7d09
2 parents 2571910 + b37ea4f
commit 71c7d09
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":