Remove ibc callbacks for accounts. (#277)

* Added verifiedsender to ibc-callback

* Added test

* Removed user initiated callbacks

* formatting [skip ci]

Author: Kayanski

framework/contracts/native/ibc-client/src/commands.rs

@@ -1,29 +1,24 @@
 use std::str::FromStr;
 
 use abstract_sdk::{
+    feature_objects::{AnsHost, VersionControlContract},
     features::AccountIdentification,
-    std::{
-        ibc_client::state::{ACCOUNTS, CONFIG},
-        ibc_host::{HostAction, InternalAction},
-        objects::{ans_host::AnsHost, version_control::VersionControlContract, ChannelEntry},
-        ICS20,
-    },
     Resolve,
 };
 use abstract_std::{
-    ibc::CallbackInfo,
     ibc_client::{
-        state::{IbcInfrastructure, IBC_INFRA, REVERSE_POLYTONE_NOTE},
+        state::{IbcInfrastructure, ACCOUNTS, CONFIG, IBC_INFRA, REVERSE_POLYTONE_NOTE},
         IbcClientCallback,
     },
-    ibc_host, manager,
-    manager::ModuleInstallConfig,
-    objects::{chain_name::ChainName, AccountId, AssetEntry},
+    ibc_host::{self, HostAction, InternalAction},
+    manager::{self, ModuleInstallConfig},
+    objects::{chain_name::ChainName, AccountId, AssetEntry, ChannelEntry},
     version_control::AccountBase,
+    ICS20,
 };
 use cosmwasm_std::{
     to_json_binary, wasm_execute, Coin, CosmosMsg, Deps, DepsMut, Empty, Env, IbcMsg, MessageInfo,
-    QueryRequest, Storage,
+    Storage,
 };
 use polytone::callbacks::CallbackRequest;
 
@@ -169,38 +164,14 @@ fn send_remote_host_action(
     Ok(note_message.into())
 }
 
-/// Perform a ICQ on a remote chain
-fn send_remote_host_query(
-    deps: Deps,
-    host_chain: ChainName,
-    queries: Vec<QueryRequest<Empty>>,
-    callback_request: CallbackRequest,
-) -> IbcClientResult<CosmosMsg<Empty>> {
-    // Send this message via the Polytone infra
-    let note_contract = IBC_INFRA.load(deps.storage, &host_chain)?.polytone_note;
-
-    let note_message = wasm_execute(
-        note_contract.to_string(),
-        &polytone_note::msg::ExecuteMsg::Query {
-            msgs: queries,
-            callback: callback_request,
-            timeout_seconds: PACKET_LIFETIME.into(),
-        },
-        vec![],
-    )?;
-
-    Ok(note_message.into())
-}
-
 /// Sends a packet with an optional callback.
 /// This is the top-level function to do IBC related actions.
 pub fn execute_send_packet(
     deps: DepsMut,
-    env: Env,
+    _env: Env,
     info: MessageInfo,
     host_chain: String,
     action: HostAction,
-    callback_info: Option<CallbackInfo>,
 ) -> IbcClientResult {
     let host_chain = ChainName::from_str(&host_chain)?;
 
@@ -219,44 +190,18 @@ pub fn execute_send_packet(
         return Err(IbcClientError::ForbiddenInternalCall {});
     }
 
-    let callback_request = callback_info.map(|c| CallbackRequest {
-        receiver: env.contract.address.to_string(),
-        msg: to_json_binary(&IbcClientCallback::UserRemoteAction(c)).unwrap(),
-    });
-
     let note_message = send_remote_host_action(
         deps.as_ref(),
         account_id,
         account_base,
         host_chain,
         action,
-        callback_request,
+        None,
     )?;
 
     Ok(IbcClientResponse::action("handle_send_msgs").add_message(note_message))
 }
 
-// Top-level function for performing queries.
-pub fn execute_send_query(
-    deps: DepsMut,
-    env: Env,
-    host_chain: String,
-    queries: Vec<QueryRequest<Empty>>,
-    callback_info: CallbackInfo,
-) -> IbcClientResult {
-    let host_chain = ChainName::from_str(&host_chain)?;
-
-    let callback_request = CallbackRequest {
-        receiver: env.contract.address.to_string(),
-        msg: to_json_binary(&IbcClientCallback::UserRemoteAction(callback_info)).unwrap(),
-    };
-
-    let note_message =
-        send_remote_host_query(deps.as_ref(), host_chain, queries, callback_request)?;
-
-    Ok(IbcClientResponse::action("handle_send_msgs").add_message(note_message))
-}
-
 /// Registers an Abstract Account on a remote chain.
 pub fn execute_register_account(
     deps: DepsMut,