Skip to content
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

--memo-path of ibc-transfer is confusing with Namada transaction memo --memo #3517

Closed
yito88 opened this issue Jul 16, 2024 · 0 comments · Fixed by #3519
Closed

--memo-path of ibc-transfer is confusing with Namada transaction memo --memo #3517

yito88 opened this issue Jul 16, 2024 · 0 comments · Fixed by #3519
Assignees

Comments

@yito88
Copy link
Member

yito88 commented Jul 16, 2024

We can rename it and split into two options like

  • --ibc-shielding-data: Set MASP proofs for IBC shielding transfer
  • --ibc-memo: Set the string to the ICS-20/ICS-721 packet memo field

It should fail when both are set because Both use the IBC packet memo field.
(--ibc-shielding-data would be used only for shielding transfer between Namada chains.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant