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

Remove or rename ICA 'BindPort' function #289

Closed
3 tasks
colin-axner opened this issue Jul 22, 2021 · 1 comment
Closed
3 tasks

Remove or rename ICA 'BindPort' function #289

colin-axner opened this issue Jul 22, 2021 · 1 comment
Labels
27-interchain-accounts good first issue Good for newcomers
Milestone
Interchain Accounts

Comments

@colin-axner
Copy link
Contributor

Summary

It binds the port and claims the capability. InitInterchainAccount does the same functionality without calling that function. It should either be renamed to BindPortAndClaimCapability and used, or removed, or it should only bind to the port

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@colin-axner colin-axner added this to the Interchain Accounts milestone Jul 22, 2021
@colin-axner colin-axner mentioned this issue Jul 22, 2021
Merged
9 tasks
@colin-axner colin-axner added the good first issue Good for newcomers label Jul 26, 2021
@damiannolan damiannolan mentioned this issue Aug 31, 2021
Merged
9 tasks
@colin-axner
Copy link
Contributor Author

closed by #362

@crodriguezvega crodriguezvega moved this to Done in ibc-go Dec 30, 2021
faddat pushed a commit to notional-labs/ibc-go that referenced this issue Mar 1, 2022
@rsum1
Merge PR cosmos#289: Update testnet server setup
ffc10f4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
27-interchain-accounts good first issue Good for newcomers
Projects
ibc-go
Archived in project
Milestone
Interchain Accounts
Development

No branches or pull requests
2 participants
@crodriguezvega @colin-axner