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

Update ADR 010 to include tx raw commands #2383

Closed
5 tasks
ljoss17 opened this issue Jul 8, 2022 · 0 comments · Fixed by #2382
Closed
5 tasks

Update ADR 010 to include tx raw commands #2383

ljoss17 opened this issue Jul 8, 2022 · 0 comments · Fixed by #2382
Assignees
@ljoss17
Milestone
v1.0.0

Comments

@ljoss17
Copy link
Contributor

ljoss17 commented Jul 8, 2022

Summary

Updating the ADR 010 to include the tx raw commands.

Problem Definition

The ADR 010 doesn't have the tx raw commands, which require changes in order to have a similar syntax as the other commands.

Proposal

  • Adding the tx raw commands
  • Removing the raw subcommand
  • Updating the flags in order to replace outdated prefixes --src- and --dst-

Acceptance Criteria

ADR 010 complete with the new tx raw commands.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@ljoss17 ljoss17 mentioned this issue Jul 8, 2022
Merged
6 tasks
@ljoss17 ljoss17 self-assigned this Jul 8, 2022
@romac romac moved this to Backlog in IBC-rs: the road to v1 Jul 13, 2022
@romac romac added this to the v1.0.0 milestone Jul 13, 2022
@ljoss17 ljoss17 mentioned this issue Jul 15, 2022
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ljoss17 ljoss17

Labels
None yet
Projects
No open projects
IBC-rs: the road to v1
Status: Backlog
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

Update ADR 010 to include tx raw commands informalsystems/hermes
2 participants
@romac @ljoss17