-
Notifications
You must be signed in to change notification settings - Fork 65
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
[Tunnel] implement route - Axelar #574
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.
branch is conflicted, please help fix
// axelar_ibc_channel specifies the IBC channel used by the tunnel to communicate with the axelar chain. | ||
string axelar_ibc_channel = 8 [(gogoproto.customname) = "AxelarIBCChannel"]; | ||
// axelar_gmp_account is the account address on axelar chain that processes and verifies Axelar GMP transactions. | ||
string axelar_gmp_account = 9; |
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.
should it be renamed to AxelarGMPAccount (based on the comment above)? If not neccesary, can be resolved.
) | ||
|
||
var ( | ||
packetABI, _ = abi.NewType("tuple", "", []abi.ArgumentMarshaling{ |
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.
this is for Axelar only or for the others (in the future) as well? b/c i see you rename tss part (to have tss prefix), should it be align?
Implementation details
Integration test
Please ensure the following requirements are met before submitting a pull request:
CHANGELOG_UNRELEASED.md
Files changed
tab in the Github PR explorer)