fix check for SyftError when updating server setting on notification …

…enable/disable
OpenMined · Jul 25, 2024 · f38132e · f38132e
1 parent bf76642
commit f38132e
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/packages/syft/src/syft/service/notifier/notifier_service.py b/packages/syft/src/syft/service/notifier/notifier_service.py
@@ -208,8 +208,8 @@ def turn_on(
 
         settings_service = context.server.get_service("settingsservice")
         result = settings_service.update(context, notifications_enabled=True)
-        if result.is_err():
-            logger.info(f"Failed to update Server Settings: {result.err()}")
+        if isinstance(result, SyftError):
+            logger.info(f"Failed to update Server Settings: {result.message}")
 
         return SyftSuccess(message="Notifications enabled successfully.")
 
@@ -235,8 +235,8 @@ def turn_off(
 
         settings_service = context.server.get_service("settingsservice")
         result = settings_service.update(context, notifications_enabled=False)
-        if result.is_err():
-            logger.info(f"Failed to update Server Settings: {result.err()}")
+        if isinstance(result, SyftError):
+            logger.info(f"Failed to update Server Settings: {result.message}")
 
         return SyftSuccess(message="Notifications disabled succesfullly")