From 48127225b2d3a3a278ee27cc3cb38f1392f5a030 Mon Sep 17 00:00:00 2001
From: Carlos Rodriguez <crodveg@gmail.com>
Date: Tue, 15 Mar 2022 13:06:39 +0100
Subject: [PATCH 1/2] fixes for documentation

---
 docs/apps/interchain-accounts/auth-modules.md | 27 ++++++++++++++-----
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/docs/apps/interchain-accounts/auth-modules.md b/docs/apps/interchain-accounts/auth-modules.md
index b87265d4e28..159cf34c312 100644
--- a/docs/apps/interchain-accounts/auth-modules.md
+++ b/docs/apps/interchain-accounts/auth-modules.md
@@ -220,8 +220,13 @@ If the controller chain is connected to a host chain using the host module on ib
 
 Begin by unmarshaling the acknowledgement into sdk.TxMsgData:
 ```go
+var ack channeltypes.Acknowledgement
+if err := channeltypes.SubModuleCdc.UnmarshalJSON(acknowledgement, &ack); err != nil {
+    return err
+}
+
 txMsgData := &sdk.TxMsgData{}
-if err := proto.Unmarshal(ack.Acknowledgement(), txMsgData); err != nil {
+if err := proto.Unmarshal(ack.GetResult(), txMsgData); err != nil {
     return err
 }
 ```
@@ -232,6 +237,8 @@ The auth module should interpret the txMsgData.Data as follows:
 ```go
 switch len(txMsgData.Data) {
 case 0:
+    // see documentation below for SDK 0.46.x or greater
+default:
     for _, msgData := range txMsgData.Data {
         if err := handler(msgData); err != nil {
             return err
@@ -246,8 +253,14 @@ A router could be used, or more simply a switch statement.
 
 ```go
 func handler(msgData sdk.MsgData) error {
-switch msgData.TypeURL {
-case banktypes.MsgSend:
+sdkMsgs := []sdk.Msg{
+	&banktypes.MsgSend{}, 
+	&stakingtypes.MsgDelegate{}, 
+	&transfertypes.MsgTransfer{},
+}
+
+switch msgData.MsgType {
+case sdk.MsgTypeURL(sdkMsgs[0]):
     msgResponse := &banktypes.MsgSendResponse{}
     if err := proto.Unmarshal(msgData.Data, msgResponse}; err != nil {
         return err
@@ -255,7 +268,7 @@ case banktypes.MsgSend:
 
     handleBankSendMsg(msgResponse)
 
-case stakingtypes.MsgDelegate:
+case sdk.MsgTypeURL(sdkMsgs[1]):
     msgResponse := &stakingtypes.MsgDelegateResponse{}
     if err := proto.Unmarshal(msgData.Data, msgResponse}; err != nil {
         return err
@@ -263,7 +276,7 @@ case stakingtypes.MsgDelegate:
 
     handleStakingDelegateMsg(msgResponse)
 
-case transfertypes.MsgTransfer:
+case sdk.MsgTypeURL(sdkMsgs[2]):
     msgResponse := &transfertypes.MsgTransferResponse{}
     if err := proto.Unmarshal(msgData.Data, msgResponse}; err != nil {
         return err
@@ -281,8 +294,8 @@ The auth module should interpret the txMsgData.Responses as follows:
 
 ```go
 ...
-// switch statement from above continued
-default:
+// switch statement from above
+case 0:
     for _, any := range txMsgData.MsgResponses {
         if err := handleAny(any); err != nil {
             return err

From cf7a3ff537051ae2de8f56c7bcd8653e631263e3 Mon Sep 17 00:00:00 2001
From: Carlos Rodriguez <carlos@interchain.io>
Date: Tue, 15 Mar 2022 14:49:58 +0100
Subject: [PATCH 2/2] review comment

---
 docs/apps/interchain-accounts/auth-modules.md | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/docs/apps/interchain-accounts/auth-modules.md b/docs/apps/interchain-accounts/auth-modules.md
index 159cf34c312..0e8738f50c0 100644
--- a/docs/apps/interchain-accounts/auth-modules.md
+++ b/docs/apps/interchain-accounts/auth-modules.md
@@ -253,14 +253,8 @@ A router could be used, or more simply a switch statement.
 
 ```go
 func handler(msgData sdk.MsgData) error {
-sdkMsgs := []sdk.Msg{
-	&banktypes.MsgSend{}, 
-	&stakingtypes.MsgDelegate{}, 
-	&transfertypes.MsgTransfer{},
-}
-
 switch msgData.MsgType {
-case sdk.MsgTypeURL(sdkMsgs[0]):
+case sdk.MsgTypeURL(&banktypes.MsgSend{}):
     msgResponse := &banktypes.MsgSendResponse{}
     if err := proto.Unmarshal(msgData.Data, msgResponse}; err != nil {
         return err
@@ -268,7 +262,7 @@ case sdk.MsgTypeURL(sdkMsgs[0]):
 
     handleBankSendMsg(msgResponse)
 
-case sdk.MsgTypeURL(sdkMsgs[1]):
+case sdk.MsgTypeURL(&stakingtypes.MsgDelegate{}):
     msgResponse := &stakingtypes.MsgDelegateResponse{}
     if err := proto.Unmarshal(msgData.Data, msgResponse}; err != nil {
         return err
@@ -276,7 +270,7 @@ case sdk.MsgTypeURL(sdkMsgs[1]):
 
     handleStakingDelegateMsg(msgResponse)
 
-case sdk.MsgTypeURL(sdkMsgs[2]):
+case sdk.MsgTypeURL(&transfertypes.MsgTransfer{}):
     msgResponse := &transfertypes.MsgTransferResponse{}
     if err := proto.Unmarshal(msgData.Data, msgResponse}; err != nil {
         return err