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

bring in changes from main branch to v3 #366

Merged
merged 12 commits into from
Jan 9, 2023
Merged

bring in changes from main branch to v3 #366

merged 12 commits into from
Jan 9, 2023

Conversation

jtieri
Copy link
Contributor

@jtieri jtieri commented Jan 7, 2023

The v3 branch was missing a few changes from the main branch like the fix for building the binary, some changes to the poller, and the changes for the packet forward middleware. This PR also bumps the ibc-go version to v3.4.0

DavidNix and others added 10 commits January 6, 2023 14:45
* Make configutil public

* Rename test package to chainutil. Make util packages subpackages of util package

* Consolidate into single testutil package
…orwards (#306)

* Make examples its own module

* Make cosmos specific module and run in CI in separate task

* Update e2e test for memo refactor

* Update test to chain-level params

* Use gaia with pfm with configurable timeout and retries

* Update SendIBCTransfer uses

* fix interchain test

* Add GetClients method to Relayer and helper for getting transfer channel between chains

* Update packet forward test for multi-hop, add multi-hop refund test

* Update tests for atomic forwards

* Wait for a block after ack before checking balances

* reduce wait to 1 block

* Add multi-hop flow with refund through chain with native denom. Add assertions for escrow accounts

* Remove stale comment

* handle feedback

* Add TransferOptions
@jtieri jtieri requested a review from a team as a code owner January 7, 2023 18:29
@jtieri jtieri requested a review from mark-rushakoff January 7, 2023 18:29
@jtieri
Copy link
Contributor Author

jtieri commented Jan 9, 2023

CI job for the example test cases was failing due to using the latest tag for the interchain accounts demo. We'll need to look into this and update the test case to work with the most recent changes but that is outside the scope of this PR.

I opened #367 so we can come back to this

@jtieri jtieri merged commit 11355c5 into v3 Jan 9, 2023
@jtieri jtieri deleted the justin/update-v3 branch January 9, 2023 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants