-
Notifications
You must be signed in to change notification settings - Fork 101
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(vara,gear-bank): Split gas and fees to author/treasury #3652
base: master
Are you sure you want to change the base?
Conversation
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.
If I'm not mistaken, we were talking about gas fees, not tx-s (or not only tx-s).
And I'd make this constant customisable with governance not only on upgrades.
Furthermore, this question is still considering, so pls don't merge yet.
Is it ready for review? |
Not yet. Need final touches and tests |
Fix tests to pass CI pls @ukint-vs |
To be reviewed once fetched master |
@ukint-vs pls finalise this one |
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.
On gear bank side, please perform ratio multiplying only once before transfer
Otherwise, great done 👍🏻
); | ||
assert_eq!(Balances::free_balance(alice.to_account_id()), STASH); | ||
|
||
DealWithFees::on_unbalanceds(vec![fee, tip].into_iter()); |
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.
shouldn't the case be covered by pallet transaction payment?
Actualise branch and resolve opened discussions, please @ukint-vs |
SplitGas
andSplitTxFeeRatio
in gear-bank config@gear-tech/dev