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.

Sign up for GitHub

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

Jump to bottom

Rename PacketData to Payload #7417

Closed
4 tasks
chatton opened this issue Oct 8, 2024 · 0 comments
Closed
4 tasks

Rename PacketData to Payload #7417

chatton opened this issue Oct 8, 2024 · 0 comments
Assignees
@bznein
Labels
feat: ibc-eureka

Comments

@chatton
Copy link
Contributor

chatton commented Oct 8, 2024

Summary of Bug

Currently we have a PacketData struct, which has an embedded Payload type. It is more in line with the spec if we create a single type Payload that has all of the fields, i.e.

Source/DestPorts, encoding, version, application bz.

ref #7413 (comment)

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
  • Estimate provided
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bznein bznein

Labels
feat: ibc-eureka
Projects
IBC
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chatton @bznein