Skip to content

Fix for API change on chain#120

Merged
n13 merged 3 commits into
mainfrom
fix/msig-approve-call-binding
Jul 24, 2026
Merged

Fix for API change on chain#120
n13 merged 3 commits into
mainfrom
fix/msig-approve-call-binding

Conversation

@n13

@n13 n13 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Adapt to the chain's multisig approve call-binding change (Quantus-Network/chain#622): Multisig.approve now requires the proposal's encoded inner call as a third parameter, which must match the stored payload.

  • Regenerate metadata + subxt codegen from the latest runtime (spec 135 / tx 3; compat pair updated in src/config/mod.rs)
  • multisig approve and approve_proposal() now fetch the proposal's stored call and resubmit it with the approval
  • Update exercise multisig scenario accordingly
  • Rename verify_aggregated_proofverify_private_batch at call sites (wormhole rename from chain#618, pulled in by the metadata regen)

Tested end-to-end against a local dev node: create → propose → approve → execute, plus quantus exercise --phases multisig (3/3 passing).

n13 and others added 3 commits July 23, 2026 15:03
multisig approve now has call binding
Keep this branch's regenerated metadata (multisig approve call binding /
tx 3) and fold in main's public-proof-agg CLI changes. Compatibility
table includes 134/2, 135/2, and 135/3.

Co-authored-by: Cursor <[email protected]>
@n13
n13 merged commit 6489fd6 into main Jul 24, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants