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

portkeepers should be able to handle adding their routes to ibcrouter #1449

Closed
3 tasks
charleenfei opened this issue May 27, 2022 · 2 comments
Closed
3 tasks
Labels
type: refactor Architecture, code or CI improvements that may or may not tackle technical debt.

Comments

@charleenfei
Copy link
Contributor

charleenfei commented May 27, 2022

Summary of Bug

ideally the app dev doesnt need to call ibcRouter.AddRoute(xyz) or instantiate the NewRouter (ibcRouter type) for each module.

just adding this here for visibility/to simplify app wiring, @AdityaSripal you may already have a solution in mind

Version

Steps to Reproduce


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@charleenfei
Copy link
Contributor Author

just discussed with @AdityaSripal , we should add a batch of work to the channel upgradeability sprint to revisit the ibc router as a whole, especially in light of the new smart contract ibc use cases

@crodriguezvega crodriguezvega added the type: refactor Architecture, code or CI improvements that may or may not tackle technical debt. label Jan 19, 2024
@colin-axner
Copy link
Contributor

closing for lack of concrete issue/proposal outline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: refactor Architecture, code or CI improvements that may or may not tackle technical debt.
Projects
None yet
Development

No branches or pull requests

3 participants