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: bonds #645

Merged
merged 35 commits into from
Aug 11, 2023
Merged

feat: bonds #645

merged 35 commits into from
Aug 11, 2023

Conversation

Roznovjak
Copy link
Contributor

No description provided.

@Roznovjak Roznovjak self-assigned this Jul 19, 2023
@github-actions
Copy link

github-actions bot commented Jul 19, 2023

New crates:

  • pallet-bonds: v1.0.0

Crate versions that have been updated:

  • runtime-integration-tests: v1.9.0 -> v1.9.1
  • pallet-asset-registry: v2.2.4 -> v2.3.0
  • pallet-circuit-breaker: v1.1.15 -> v1.1.16
  • pallet-dca: v1.1.7 -> v1.1.8
  • pallet-liquidity-mining: v4.2.3 -> v4.2.4
  • pallet-omnipool: v3.2.0 -> v3.2.1
  • pallet-omnipool-liquidity-mining: v2.0.10 -> v2.0.11
  • pallet-otc: v1.0.1 -> v1.0.2
  • pallet-stableswap: v2.1.1 -> v2.1.2
  • pallet-xcm-rate-limiter: v0.1.0 -> v0.1.1
  • primitives: v5.8.2 -> v5.8.3
  • hydradx-adapters: v0.5.0 -> v0.5.1
  • hydradx-runtime: v171.0.0 -> v172.0.0
  • hydradx-traits: v2.5.0 -> v2.5.1

Runtime version has been increased.

@codecov
Copy link

codecov bot commented Jul 20, 2023

Codecov Report

Patch coverage: 78.35% and project coverage change: +0.13% 🎉

Comparison is base (aecefb9) 68.08% compared to head (3a2507f) 68.22%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #645      +/-   ##
==========================================
+ Coverage   68.08%   68.22%   +0.13%     
==========================================
  Files         153      156       +3     
  Lines       11692    11789      +97     
==========================================
+ Hits         7961     8043      +82     
- Misses       3731     3746      +15     
Files Changed Coverage Δ
primitives/src/constants.rs 0.00% <ø> (ø)
runtime/hydradx/src/lib.rs 3.24% <0.00%> (-0.05%) ⬇️
runtime/hydradx/src/weights/bonds.rs 0.00% <0.00%> (ø)
traits/src/registry.rs 0.00% <ø> (ø)
pallets/asset-registry/src/types.rs 61.53% <71.42%> (+61.53%) ⬆️
pallets/asset-registry/src/lib.rs 85.46% <75.00%> (+1.53%) ⬆️
pallets/bonds/src/lib.rs 83.67% <83.67%> (ø)
pallets/bonds/src/benchmarks.rs 100.00% <100.00%> (ø)
runtime/hydradx/src/assets.rs 83.33% <100.00%> (+1.33%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Roznovjak Roznovjak marked this pull request as ready for review July 28, 2023 10:08
amount: T::Balance,
maturity: Moment,
) -> DispatchResult {
let who = T::IssueOrigin::ensure_origin(origin)?;
Copy link
Member

Choose a reason for hiding this comment

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

anyone can issue bonds

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok. I will remove the config param. The idea was to have it configurable, but configure it so that everyone can do that.

Copy link
Member

Choose a reason for hiding this comment

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

no need to configure it if anyone can do it

Copy link
Member

Choose a reason for hiding this comment

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

but we could keep it if we plan to enable this for others only later

pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
integration-tests/src/bonds.rs Outdated Show resolved Hide resolved
integration-tests/src/bonds.rs Outdated Show resolved Hide resolved
@mrq1911 mrq1911 changed the title feat: Bonds pallet feat: bonds Aug 8, 2023
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved

// Bonds
parameter_types! {
pub ProtocolFee: Permill = Permill::from_percent(2);
Copy link
Member

Choose a reason for hiding this comment

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

this fee is rather large... how it was determined?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh. I just needed to use some value. We need to discuss and determine what value to use.

@@ -0,0 +1,76 @@
// This file is part of HydraDX.
Copy link
Member

Choose a reason for hiding this comment

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

has to be benchmarked

pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
pallets/bonds/src/lib.rs Outdated Show resolved Hide resolved
@mrq1911 mrq1911 merged commit 07156fc into master Aug 11, 2023
8 checks passed
@Roznovjak Roznovjak deleted the bonds branch August 12, 2023 10:59
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.

4 participants