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

Add support for IBC transfers #127

Closed
webmaster128 opened this issue Jun 1, 2023 · 0 comments · Fixed by #128
Closed

Add support for IBC transfers #127

webmaster128 opened this issue Jun 1, 2023 · 0 comments · Fixed by #128
Assignees

Comments

@webmaster128
Copy link
Member

The message looks like this:

export interface MsgTransfer {
  /** the port on which the packet will be sent */
  sourcePort: string;
  /** the channel by which the packet will be sent */
  sourceChannel: string;
  /** the tokens to be transferred */
  token?: Coin;
  /** the sender address */
  sender: string;
  /** the recipient address on the destination chain */
  receiver: string;
  /**
   * Timeout height relative to the current block height.
   * The timeout is disabled when set to 0.
   */
  timeoutHeight?: Height;
  /**
   * Timeout timestamp in absolute nanoseconds since unix epoch.
   * The timeout is disabled when set to 0.
   */
  timeoutTimestamp: Long;
  /** optional memo */
  memo: string;
}

Some notes here:

  • sourcePort should be pre-set to "transfer" by default, but can be changed by the user
  • token is required and an aribrary numeric amount and arbitrary string denom (only one, no list here)
  • We can use timeoutTimestamp only and put a date in the future. timeoutHeight can be set to undefined. Ideally we offer the user simple options like 12 hours from now, 1 day from now, 2 days from now, 3 days from now, 7 days from now, 10 days from now, 2 weeks from now, 3 weeks from now, 1 month from now, custom time
  • The memo in the message is a different memo than the transaction memo. Both can be set individually.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants