node
node copied to clipboard
Separate messages for each required policy group
https://github.com/zeta-chain/node/pull/1552#discussion_r1449091703
For simplicity and better maintainability, each message should have a single policy group requirement.
We should change it for MsgUpdateCrosschainFlags and list other message with different policy group requirements
We can refactor into the following messages. cc @lumtis
// MsgDisableCCTXflags -> PolicyType_groupEmergency
// Disable inbound and outbound transactions , Disable both or any one
// MsgEnableCCTXflags -> PolicyType_groupOperational
// Enable inbound and outbound transactions , Enable both or any one
// MsgUpdateGasPriceFlags -> PolicyType_groupOperational . Update values
// Update gas price flags
Yes, this can looks like this.
Broke this down into issue #2244 and #2245
Closing this issue , and working on the new issues listed above