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!: update fee calculation #289

Merged
merged 3 commits into from
Jul 2, 2021
Merged

feat!: update fee calculation #289

merged 3 commits into from
Jul 2, 2021

Conversation

Roznovjak
Copy link
Contributor

Description

Current fees for a TX are set too low. Tune the costs.

@auto-add-label auto-add-label bot added the type/enhancement New feature or request label Jun 29, 2021
@Roznovjak
Copy link
Contributor Author

Transfer costs 0.0047 $ (the same as in Basilisk)
Full block: 395 $ (393 in Basilisk)
Fee multiplier after one hour of operation with fully loaded blocks: 1.0135 (the same as in Basilisk)
After one day: 1.3826 (the same as in Basilisk)

@Roznovjak Roznovjak requested review from jak-pan and mrq1911 June 29, 2021 12:37
@jak-pan jak-pan changed the title feat: update fee calculation feat!: update fee calculation Jun 30, 2021
@auto-add-label auto-add-label bot added 00-breaking-change and removed type/enhancement New feature or request labels Jun 30, 2021
@codecov
Copy link

codecov bot commented Jul 2, 2021

Codecov Report

Merging #289 (6a5bad0) into master (00c53a4) will increase coverage by 0.45%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #289      +/-   ##
==========================================
+ Coverage   14.82%   15.27%   +0.45%     
==========================================
  Files          24       20       -4     
  Lines        6970     6776     -194     
==========================================
+ Hits         1033     1035       +2     
+ Misses       5937     5741     -196     
Impacted Files Coverage Δ
pallets/xyk/src/lib.rs 84.01% <ø> (+2.80%) ⬆️
pallets/claims/src/weights.rs
pallets/transaction-multi-payment/src/weights.rs
pallets/xyk/src/weights.rs
pallets/exchange/src/weights.rs

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e634d4e...6a5bad0. Read the comment docs.

@Roznovjak Roznovjak merged commit d1298bd into master Jul 2, 2021
@Roznovjak Roznovjak deleted the feat/adjust_fee branch July 2, 2021 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants