-
Notifications
You must be signed in to change notification settings - Fork 38
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 custom wasm cli #240
Add custom wasm cli #240
Conversation
please wait for me to test, testing shows tax is not deduced from wallet |
ok, tested that it will deduce tax |
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! just a comment if you want to remove custom/wasm/simulation and custom/wasm/client/rest
Let's not remove it. The simulation needs to take care of the taxes too. That's why we need to register Weighted operations in module.go. |
Summary of changes
AppModuleBasic
to override REST handlers and CLI commands.feeutils.go
#238