Merge pull request #8841 from OpenMined/aziz/request_msg

Update request message to indicate function name
OpenMined · May 22, 2024 · 8a28ad6 · 8a28ad6
2 parents 55036c8 + aa700c4
commit 8a28ad6
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/packages/syft/src/syft/service/request/request.py b/packages/syft/src/syft/service/request/request.py
@@ -570,11 +570,14 @@ def approve(
             )
         if message and metadata and metadata.show_warnings and not disable_warnings:
             prompt_warning_message(message=message, confirm=True)
+        msg = (
+            "Approving request ",
+            f"on change {self.code.service_func_name} " if is_code_request else "",
+            f"for domain {api.node_name}",
+        )
 
-        print(f"Approving request for domain {api.node_name}")
+        print("".join(msg))
         res = api.services.request.apply(self.id, **kwargs)
-        # if isinstance(res, SyftSuccess):
-
         return res
 
     def deny(self, reason: str) -> SyftSuccess | SyftError: