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 memKey from Interchain Accounts #336

Closed
3 tasks
colin-axner opened this issue Aug 13, 2021 · 1 comment
Closed
3 tasks

Remove memKey from Interchain Accounts #336

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

Comments

@colin-axner
Copy link
Contributor

Summary

ICA doesn't need a memstore. Typically only the capability keeper needs a mem store. The memkey can be removed from the ICA keeper

For Admin Use

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

closed by #342

@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
@michaelfig
fix: update nchainz and configs for cosmos v0.40.0-rc3 (cosmos#336)
f8acf82
faddat pushed a commit to notional-labs/ibc-go that referenced this issue Mar 1, 2022
@michaelfig @colin-axner
fix: update nchainz and configs for cosmos v0.40.0-rc3 (cosmos#336)
21c64e6
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