Overview
configure_approval_rule and create_transfer_request (contracts/multisig-transfer/src/lib.rs) — worth dedicated tests for configuring a rule that requires zero approvers (should presumably be rejected as meaningless), reconfiguring an existing rule while a transfer request under the old rule is still pending, and creating a request for a category with no configured rule at all.
Tasks
Acceptance Criteria
Notes for Contributors
Requires Rust + Soroban. Workspace: contracts/ (soroban-sdk 23, members: assetsup, contrib, multisig-wallet, multisig-transfer, asset-maintenance). Run cargo test --all, cargo fmt --all -- --check, and cargo clippy --all-targets --all-features -- -D warnings before opening a PR — CI enforces all three. Comment below to be assigned.
Overview
configure_approval_ruleandcreate_transfer_request(contracts/multisig-transfer/src/lib.rs) — worth dedicated tests for configuring a rule that requires zero approvers (should presumably be rejected as meaningless), reconfiguring an existing rule while a transfer request under the old rule is still pending, and creating a request for a category with no configured rule at all.Tasks
configure_approval_rulerejects a zero-required-approvers configuration.Acceptance Criteria
Notes for Contributors
Requires Rust + Soroban. Workspace:
contracts/(soroban-sdk 23, members: assetsup, contrib, multisig-wallet, multisig-transfer, asset-maintenance). Runcargo test --all,cargo fmt --all -- --check, andcargo clippy --all-targets --all-features -- -D warningsbefore opening a PR — CI enforces all three. Comment below to be assigned.