apply some comments

zetaclient/authz/authz_signer.go

@@ -1,4 +1,5 @@
 // Package authz provides a signer object for transactions using grants
+// grants are used to allow a hotkey to sign transactions on behalf of the observers
 package authz
 
 import (

zetaclient/chains/bitcoin/observer/observer.go

@@ -53,7 +53,6 @@ const (
 var _ interfaces.ChainObserver = &Observer{}
 
 // Logger contains list of loggers used by Bitcoin chain observer
-// TODO(revamp): Merge this logger with the one in evm
 // https://github.com/zeta-chain/node/issues/2022
 type Logger struct {
 	// Chain is the parent logger for the chain

zetaclient/chains/bitcoin/observer/outbound.go

@@ -332,8 +332,8 @@ func (ob *Observer) refreshPendingNonce() {
 	}
 }
 
-// getOutboundIDByNonce gets the outbound ID from the nonce of the outbound transaction.
-// test is true for unit test only.
+// getOutboundIDByNonce gets the outbound ID from the nonce of the outbound transaction
+// test is true for unit test only
 func (ob *Observer) getOutboundIDByNonce(nonce uint64, test bool) (string, error) {
 
 	// There are 2 types of txids an observer can trust

zetaclient/chains/evm/observer/observer.go

@@ -558,7 +558,7 @@ func (ob *Observer) PostGasPrice() error {
 }
 
 // TransactionByHash query transaction by hash via JSON-RPC
-// TODO(revamp): move to a tx file
+// TODO(revamp): update this method as a pure RPC method that takes two parameters (jsonRPC, and txHash) and move to upper package to file rpc.go
 func (ob *Observer) TransactionByHash(txHash string) (*ethrpc.Transaction, bool, error) {
 	tx, err := ob.evmJSONRPC.EthGetTransactionByHash(txHash)
 	if err != nil {

zetaclient/zetacore/query.go

@@ -270,19 +270,6 @@ func (c *Client) GetNodeInfo() (*tmservice.GetNodeInfoResponse, error) {
 	return nil, err
 }
 
-// GetLastBlockHeightByChain returns the last block height by chain
-func (c *Client) GetLastBlockHeightByChain(chain chains.Chain) (*crosschaintypes.LastBlockHeight, error) {
-	client := crosschaintypes.NewQueryClient(c.grpcConn)
-	resp, err := client.LastBlockHeight(
-		context.Background(),
-		&crosschaintypes.QueryGetLastBlockHeightRequest{Index: chain.ChainName.String()},
-	)
-	if err != nil {
-		return nil, err
-	}
-	return resp.LastBlockHeight, nil
-}
-
 // GetBlockHeight returns the block height
 func (c *Client) GetBlockHeight() (int64, error) {
 	client := crosschaintypes.NewQueryClient(c.grpcConn)

zetaclient/zetacore/query_test.go

@@ -461,30 +461,6 @@ func TestZetacore_GetNodeInfo(t *testing.T) {
 	require.Equal(t, expectedOutput, *resp)
 }
 
-func TestZetacore_GetLastBlockHeightByChain(t *testing.T) {
-	index := chains.BscMainnet
-	expectedOutput := crosschainTypes.QueryGetLastBlockHeightResponse{
-		LastBlockHeight: &crosschainTypes.LastBlockHeight{
-			Index:              index.ChainName.String(),
-			Chain:              "7000",
-			LastOutboundHeight: 2134123,
-			LastInboundHeight:  1234333,
-		},
-	}
-	input := crosschainTypes.QueryGetLastBlockHeightRequest{Index: index.ChainName.String()}
-	method := "/zetachain.zetacore.crosschain.Query/LastBlockHeight"
-	server := setupMockServer(t, crosschainTypes.RegisterQueryServer, method, input, expectedOutput)
-	server.Serve()
-	defer closeMockServer(t, server)
-
-	client, err := setupZetacoreClient()
-	require.NoError(t, err)
-
-	resp, err := client.GetLastBlockHeightByChain(index)
-	require.NoError(t, err)
-	require.Equal(t, expectedOutput.LastBlockHeight, resp)
-}
-
 func TestZetacore_GetZetaBlockHeight(t *testing.T) {
 	expectedOutput := crosschainTypes.QueryLastZetaHeightResponse{Height: 12345}
 	input := crosschainTypes.QueryLastZetaHeightRequest{}

zetaclient/zetacore/tx.go

@@ -71,6 +71,7 @@ func GasPriceMultiplier(chainID int64) (float64, error) {
 }
 
 // WrapMessageWithAuthz wraps a message with an authz message
+// used since a hotkey is used to broadcast the transactions, instead of the operator
 func (c *Client) WrapMessageWithAuthz(msg sdk.Msg) (sdk.Msg, clientauthz.Signer, error) {
 	msgURL := sdk.MsgTypeURL(msg)