Skip to content

Fix/556 milestones treasury admin auth - #757

Open
sladeoj wants to merge 2 commits into
MergeFi:mainfrom
sladeoj:fix/556-milestones-treasury-admin-auth
Open

sladeoj wants to merge 2 commits into
MergeFi:mainfrom
sladeoj:fix/556-milestones-treasury-admin-auth

Conversation

@sladeoj

@sladeoj sladeoj commented Oct 21, 2026

Copy link
Copy Markdown

closes #556

Summary

Related issues

Changes

Test plan

  • Existing tests pass (make test)
  • New tests added for changed behaviour
  • Manually verified against a local node / testnet where applicable

Checklist

  • Branch is up to date with main
  • No debug / dead code left in
  • cargo fmt and cargo clippy are clean

sladeoj and others added 2 commits October 21, 2026 01:22
@sladeoj
sladeoj requested a review from chonilius as a code owner October 21, 2026 01:32
@drips-wave

drips-wave Bot commented Oct 21, 2026

Copy link
Copy Markdown

@sladeoj Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@vercel

vercel Bot commented Oct 21, 2026

Copy link
Copy Markdown

@sladeoj is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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.

milestones::set_treasury has no test proving it requires the admin's authorization

1 participant