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

refactor(templates): add all ibc commands #3858

Merged
merged 3 commits into from
Dec 21, 2023
Merged

Conversation

julienrbrt
Copy link
Member

Closes: #3857
Follow-up of #3836

@ilgooz ilgooz merged commit 3dda9b0 into main Dec 21, 2023
42 checks passed
@ilgooz ilgooz deleted the julien/add-all-ibc-cmd branch December 21, 2023 11:10
Pantani added a commit that referenced this pull request Dec 25, 2023
Pantani added a commit that referenced this pull request Jan 31, 2024
* create a method to fill other methods

* add chain methods

* simplify te logic

* spin up both chains

* add reminder

* add relayers

* create configs into the code and add a helper to setup and run chains

* fix chain config

* fix rpc ports

* fix wrong urls and missing relayer accounts

* reset relayer config and avoid port conflicts

* test relayer

* fix config port address

* draft check realyer

* use hermes realyer instead ts relayer

* fix the default config values

* fix the config flag

* add missing tx.go file by default and enable cli if autocli not exist

* add changelog

* fix wrong error pkg

* add missing imports

* only scaffold `cli/tx.go` if is a ibc module

* move tx.go.plush to right place

* add comment to cobra send packet command

* add missing ibc interfaces to chain client

* set port range

* Revert "refactor(templates): add all ibc commands (#3858)"

This reverts commit 3dda9b0.

* fix changelog

* fix ibc.go app

* query channels

* check balances

* check ibc balance

* improve test cleanup

* fix chain home and config paths

* fix log typo

* cerate the chain path before use

* remove unused const

* decrease cleanup time

* setup the remote hermes app url

* use parser.ParseExpr instead parser.ParseExpr

---------

Co-authored-by: Pantani <Pantani>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v28 ICA & IBC root commands not found
3 participants