From c2d8fbd659cfbeab836e2c8c998fe613f8f5b618 Mon Sep 17 00:00:00 2001 From: swelf Date: Mon, 25 Nov 2024 22:59:43 +0300 Subject: [PATCH] review fixes: removed commented code --- contracts/dao/neutron-chain-manager/src/msg.rs | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/contracts/dao/neutron-chain-manager/src/msg.rs b/contracts/dao/neutron-chain-manager/src/msg.rs index e7b28374..ed555dc1 100644 --- a/contracts/dao/neutron-chain-manager/src/msg.rs +++ b/contracts/dao/neutron-chain-manager/src/msg.rs @@ -209,7 +209,6 @@ impl Strategy { pub fn get_ccv_update_param_permission(&self) -> Option { match self { Strategy::AllowAll => Some(CCVUpdateParamsPermission { - // enabled: true, blocks_per_distribution_transmission: true, distribution_transmission_channel: true, provider_fee_pool_addr_str: true, @@ -387,8 +386,6 @@ pub struct GlobalfeeUpdateParamsPermission { #[derive(Eq)] #[serde(rename_all = "snake_case")] pub struct CCVUpdateParamsPermission { - // TODO: do we need to manage the param - // pub enabled: bool, pub blocks_per_distribution_transmission: bool, pub distribution_transmission_channel: bool, pub provider_fee_pool_addr_str: bool, @@ -397,7 +394,8 @@ pub struct CCVUpdateParamsPermission { pub consumer_redistribution_fraction: bool, pub historical_entries: bool, pub unbonding_period: bool, - // !!! DEPRECATED !!! soft_opt_out_threshold is deprecated. see docs/docs/adrs/adr-015-partial-set-security.md + // !!! DEPRECATED !!! soft_opt_out_threshold is deprecated. + // see https://github.com/cosmos/interchain-security/blob/main/docs/docs/adrs/adr-015-partial-set-security.md pub soft_opt_out_threshold: bool, pub reward_denoms: bool, pub provider_reward_denoms: bool,