Iya feecollector

[kimurayu45z]

• Use cosmossdk.io/math in yieldaggregator
• Abolish Legacy gov proposal for AddStrategy, and use MsgRegisterStrategy with gov authority
• Prohibit using MsgDeleteVault by vault owner. Allowing only gov authority
  • It was critical
• Add FeeCollectorAddress in Params
• Use Params.WithdrawCommissionRate
• Abolish x/params and prepare MsgUpdateParams with gov authority

[kimurayu45z]

need to check

• x/yieldaggregator/client/cli/tx_register_strategy.go
• x/yieldaggregator/client/cli/tx_update_params.go
• all tests
• this one:

        withdrawModuleCommissionFee := sdk.NewDecFromInt(withdrawAmount).Mul(params.CommissionRate).RoundInt()
    withdrawVaultCommissionFee := sdk.NewDecFromInt(withdrawAmount).Mul(vault.WithdrawCommissionRate).RoundInt()
    withdrawAmountWithoutCommission := withdrawAmount.Sub(withdrawModuleCommissionFee).Sub(withdrawVaultCommissionFee)

    if withdrawModuleCommissionFee.IsPositive() {
        feeCollector, err := sdk.AccAddressFromBech32(params.FeeCollectorAddress)
        if err != nil {
            return err
        }
        err = k.bankKeeper.SendCoins(ctx, vaultModAddr, feeCollector, sdk.NewCoins(sdk.NewCoin(principal.Denom, withdrawModuleCommissionFee)))
        if err != nil {
            return err
        }
    }

    if withdrawVaultCommissionFee.IsPositive() {
        vaultOwner, err := sdk.AccAddressFromBech32(vault.Owner)
        if err != nil {
            return err
        }
        err = k.bankKeeper.SendCoins(ctx, vaultModAddr, vaultOwner, sdk.NewCoins(sdk.NewCoin(principal.Denom, withdrawVaultCommissionFee)))
        if err != nil {
            return err
        }
    }