Added 2 TODOS for the SDK

AbstractSDK · May 15, 2024 · 4c63bb3 · 4c63bb3
1 parent b5c2880
commit 4c63bb3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/contracts/client/src/handlers/execute.rs b/contracts/client/src/handlers/execute.rs
@@ -71,6 +71,7 @@ fn receive_msg(deps: DepsMut, info: MessageInfo, msg: Message, app: App) -> Clie
     // TODO: remove print
     println!("Received message: {:?}", msg);
     // check that the message sender is the server... this requires the server to be the proper version
+    // TODO, should we have a function that is able to check against a module ID directly in the SDK ?
     let sender_module = app
         .module_registry(deps.as_ref())?
         .module_info(info.sender)
@@ -88,7 +89,6 @@ fn receive_msg(deps: DepsMut, info: MessageInfo, msg: Message, app: App) -> Clie
     let len = RECEIVED
         .keys(deps.storage, None, None, Order::Ascending)
         .count();
-
     println!("Received length: {:?}", len);
 
     Ok(app

diff --git a/contracts/server/src/handlers/execute.rs b/contracts/server/src/handlers/execute.rs
@@ -127,7 +127,9 @@ pub(crate) fn route_msg(
 
             println!("ibc_msg: {:?}", ibc_msg);
             // TODO: suggested syntax
-            // let ibc_msg = app.ibc_client().module_ibc_action(chain, module,msg, callback)
+            // let ibc_msg = app.ibc_client().module_ibc_action(chain, target_module, msg, callback)
+            // TODO: We could additionally have something like to avoid having to create the module info object
+            // let ibc_msg = app.ibc_client().self_module_ibc_action(chain, msg, callback)
 
             let ibc_client_addr = app
                 .module_registry(deps.as_ref())?