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

transaction memo is not supported by cosmosclient #3997

Open
fmorency opened this issue Mar 1, 2024 · 1 comment
Open

transaction memo is not supported by cosmosclient #3997

fmorency opened this issue Mar 1, 2024 · 1 comment
Assignees
Labels
component:cosmos-client type:enhancement Improvement or update to existing feature(s).

Comments

@fmorency
Copy link

fmorency commented Mar 1, 2024

Is your feature request related to a problem or issue you encountered? Please describe.
One is not able to set a transaction memo when using the cosmosclient package.

Describe the solution you'd like
The ability to set a transaction memo when using the cosmosclient package.

Add transaction memo support to the BankSendTx helper function as well.

Backport the fix to previous Ignite versions.

Describe alternatives you've considered
Not use cosmosclient and fallback to CosmosSDK.

@fmorency
Copy link
Author

fmorency commented Mar 1, 2024

I believe one should set the Memo in the txBuilder, e.g., txBuilder.SetMemo("my memo")

@salmad3 salmad3 added type:enhancement Improvement or update to existing feature(s). component:cosmos-client and removed needs-triage labels Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:cosmos-client type:enhancement Improvement or update to existing feature(s).
Projects
Status: Todo
Development

No branches or pull requests

3 participants