-
Notifications
You must be signed in to change notification settings - Fork 151
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
fix: support calamari parachain fee calculations #749
Conversation
Hey @grenade thanks for the PR. I'll have a run at this soon, just to verify, but from a quick glance it looks good. Youll just have to run |
thank you! i've actually not been able to prove this works yet (hence the |
Is there a public node I can test this against? |
yes. any of:
|
Could you provide me with a block number/hash that contains an extrinisic that would require a fee as well? |
nvm found a block. |
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.
lgtm
After the tests, I will merge. Just as a heads up these changes will be reflected in next weeks release. |
No description provided.