-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
ibc-transfer: enable transfer parameter #6607
Conversation
will block this PR until #6641 is merged |
Codecov Report
@@ Coverage Diff @@
## master #6607 +/- ##
==========================================
- Coverage 54.76% 54.72% -0.04%
==========================================
Files 537 539 +2
Lines 36687 36770 +83
==========================================
+ Hits 20090 20121 +31
- Misses 14958 15008 +50
- Partials 1639 1641 +2 |
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
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.
post-merge ACK
@@ -60,3 +60,13 @@ message DenomTrace { | |||
// base denomination of the relayed fungible token. | |||
string base_denom = 2; | |||
} | |||
|
|||
// Params defines the set of IBC transfer parameters. | |||
// NOTE: To prevent a single token from being transferred, set the TransfersEnabled parameter to |
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.
I think this note could be slightly more clear:
- to block any cross-chain transfers -> disable ibc-transfer param
- to block intrachain transfers for a specific denom -> disable bank param for that denom
* ibc-transfer: enable transfer parameter * param_test.go * update tests and types * gRPC test * relay logic and spec update * fix tests * unused param
Description
closes: #6591
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes