Skip to content

Conversation

@saiintbrisson
Copy link
Collaborator

This patch allows recipients to have 0 weights when permissions are updatable. This plays along recipient managers.

Closes CHAIN-138.

@linear
Copy link

linear bot commented Oct 8, 2025

@coderabbitai
Copy link

coderabbitai bot commented Oct 8, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/permission0/allow-0-weight-recipients

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Oct 8, 2025

Code Coverage

Package Line Rate Health
pallets.torus0.src 96%
pallets.permission0.src.permission 84%
pallets.governance.src 93%
pallets.emission0.src.distribute 90%
pallets.permission0.src 90%
pallets.permission0.src.ext 90%
pallets.permission0.api.src 100%
pallets.torus0.api.src 79%
pallets.emission0.src 92%
Summary 91% (3511 / 3843)

@github-actions
Copy link

github-actions bot commented Oct 8, 2025

Detailed coverage report

Copy link
Member

@functor-flow functor-flow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to go

@saiintbrisson saiintbrisson merged commit 51e7ea5 into dev Oct 8, 2025
3 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.

3 participants