Skip to content

Add fee multiplier to protocol#1161

Merged
vgeddes merged 6 commits intomainfrom
fee-multiplier
Mar 21, 2024
Merged

Add fee multiplier to protocol#1161
vgeddes merged 6 commits intomainfrom
fee-multiplier

Conversation

@vgeddes
Copy link
Copy Markdown
Collaborator

@vgeddes vgeddes commented Mar 20, 2024

Since our system is currently governed by static exchange rates, we need to add a safety buffer to the calculated fees.

This ensures that if exchange rates or gas prices fluctuate, users pay enough fees to cover costs.

Companion PR: Snowfork/polkadot-sdk#127

@vgeddes vgeddes self-assigned this Mar 20, 2024
@vgeddes vgeddes marked this pull request as draft March 20, 2024 11:02
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 20, 2024

Codecov Report

Attention: Patch coverage is 53.84615% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 77.83%. Comparing base (16229c1) to head (111ee1e).
Report is 6 commits behind head on main.

Files Patch % Lines
contracts/src/upgrades/rococo/GatewayV2.sol 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1161      +/-   ##
==========================================
- Coverage   78.92%   77.83%   -1.10%     
==========================================
  Files          13       14       +1     
  Lines         408      415       +7     
  Branches       74       76       +2     
==========================================
+ Hits          322      323       +1     
- Misses         69       75       +6     
  Partials       17       17              
Flag Coverage Δ
solidity 77.83% <53.84%> (-1.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@vgeddes vgeddes marked this pull request as ready for review March 20, 2024 14:27
Copy link
Copy Markdown
Contributor

@yrong yrong left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Copy Markdown
Contributor

@alistair-singh alistair-singh left a comment

Choose a reason for hiding this comment

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

+1

@vgeddes vgeddes merged commit cc6f123 into main Mar 21, 2024
@vgeddes vgeddes deleted the fee-multiplier branch March 21, 2024 10:07
claravanstaden pushed a commit that referenced this pull request Apr 5, 2024
* Add fee multiplier

* The base fee and exchange rate should be multiplied separately

* Add FEE_MULTIPLIER to E2E config

* Update web/packages/test/scripts/set-env.sh

Co-authored-by: Alistair Singh <[email protected]>

* Make multiplier non-linear

* Add gateway upgrade for Rococo

---------

Co-authored-by: Alistair Singh <[email protected]>
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