Skip to content

adjust fees per asset#1711

Merged
brenzi merged 3 commits intomasterfrom
ab/fee-per-asset
Mar 23, 2025
Merged

adjust fees per asset#1711
brenzi merged 3 commits intomasterfrom
ab/fee-per-asset

Conversation

@brenzi
Copy link
Copy Markdown
Collaborator

@brenzi brenzi commented Mar 23, 2025

only roughly. no price oracles used yet

also fixes bogus ETH location transformation from #1710

tests

tester IncogniteeAHP account starts with 

L1: 0.003 ETH
shield 0.001 ETH
L1: 0.002 ETH
L2: 0.000998248686514887 ETH
send 0.0001 ETH to tester. received
L2: 0.000898248686514887
(fees were charged in DOT)
unshield 0.0005
L1: 0.0025 ETH
L2: 0.000398248686514887

Now testing if fees charged in ETH
send 0.0002 ETH to account without any DOT or other assets (tester2 local)
bal 0.0002
send 0.0001 back
bal 0.000094450000000000
fee 0.00000555 ETH

@brenzi brenzi added A2-applibs Affects app-libs, i.e. runtime or stf B1-releasenotes C1-low 📌 Does not elevate a release containing this beyond "low priority" E0-breaksnothing labels Mar 23, 2025
@brenzi brenzi requested a review from clangenb March 23, 2025 14:10
@brenzi brenzi merged commit 60b67ee into master Mar 23, 2025
83 of 87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A2-applibs Affects app-libs, i.e. runtime or stf B1-releasenotes C1-low 📌 Does not elevate a release containing this beyond "low priority" E0-breaksnothing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant