Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: bonded coins pallet #726

Draft
wants to merge 36 commits into
base: develop
Choose a base branch
from
Draft

feat: bonded coins pallet #726

wants to merge 36 commits into from

Conversation

rflechtner
Copy link
Contributor

fixes KILTProtocol/ticket#3585

Draft PR for the bonded currencies pallet, part of our efforts to bring VCOs to the KILT network.

Metadata Diff to Develop Branch

Peregrine Diff
Spiritnet Diff

Checklist:

  • I have verified that the code works
    • No panics! (checked arithmetic ops, no indexing array[3] use get(3), ...)
  • I have verified that the code is easy to understand
    • If not, I have left a well-balanced amount of inline comments
  • I have left the code in a better state
  • I have documented the changes (where applicable)
    • Either PR or Ticket to update the Docs
    • Link the PR/Ticket here

@rflechtner rflechtner changed the title Feat bonded coins feat: bonded coins pallet Sep 11, 2024
Copy link
Member

@Ad96el Ad96el left a comment

Choose a reason for hiding this comment

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

Mostly picky comments. Good job so far 🚀

pallets/pallet-bonded-coins/Cargo.toml Show resolved Hide resolved
pallets/pallet-bonded-coins/Cargo.toml Outdated Show resolved Hide resolved
use frame_benchmarking::{benchmarks, impl_benchmark_test_suite, whitelisted_caller};
use frame_system::RawOrigin;

benchmarks! {
Copy link
Member

Choose a reason for hiding this comment

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

We should use the benchmarking v2 here. I know it's not the time yet. I just wanted to bring it up here

pallets/pallet-bonded-coins/src/lib.rs Outdated Show resolved Hide resolved
pallets/pallet-bonded-coins/src/lib.rs Outdated Show resolved Hide resolved
pallets/pallet-bonded-coins/src/lib.rs Outdated Show resolved Hide resolved
pallets/pallet-bonded-coins/src/lib.rs Outdated Show resolved Hide resolved
pallets/pallet-bonded-coins/src/lib.rs Outdated Show resolved Hide resolved

// TODO: apply lock if pool_details.transferable != true

Ok(().into())
Copy link
Member

Choose a reason for hiding this comment

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

can be removed with DispatchResult

pallets/pallet-bonded-coins/src/lib.rs Outdated Show resolved Hide resolved
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