Merge pull request #9324 from OpenMined/eelco/deny-reason-email-fix

fix deny reason in email
OpenMined · Sep 27, 2024 · 53bb398 · 53bb398
2 parents 3384ae0 + 6db11e3
commit 53bb398
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/syft/src/syft/service/notification/email_templates.py b/packages/syft/src/syft/service/notification/email_templates.py
@@ -687,7 +687,8 @@ def email_body(notification: "Notification", context: AuthedServiceContext) -> s
             deny_reason_or_err = request_obj.get_deny_reason(context=context)
             if deny_reason_or_err.is_err():
                 deny_reason = None
-            deny_reason = deny_reason_or_err.unwrap()
+            else:
+                deny_reason = deny_reason_or_err.unwrap()
 
             if not isinstance(deny_reason, str) or not len(deny_reason):
                 deny_reason = (