-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add delivery fees #479
Add delivery fees #479
Conversation
This reverts commit 6e672e9.
Coverage Report@@ Coverage Diff @@
## master girazoki-add-delivery-fees +/- ##
=============================================================
Coverage 64.81% 64.81% 0.00%
Files 66 66
Lines 9906 9906
=============================================================
Hits 6420 6420
Misses 3486 3486
|
return filteredEvents[0]!.data[1][0].fun.asFungible.toBigInt(); | ||
}; | ||
|
||
export const getLastSentUmpMessageFee = async (context: DevModeContext, baseDelivery: bigint, txByteFee: bigint) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really useful functions :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Specially the typescript helpers/tests addition 🥇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comment in test. Apart from that everything else look good.
Add delivery fees for XCM routers in all runtimes (except Flashbox). We use
ExpontentialPrice
from polkadot-runtime-common and we charge in the native asset for each runtime.Added a couple function in TS so that we can check that indeed the messages are correctly charged. These take the latest fees from the sent messages available in storage