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

core(deps): Upgrade ibc-hooks to IBC v8 and SDK v0.50.x #163

Closed
wants to merge 4 commits into from

Conversation

alexanderbez
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the ibc-hooks Label for items related to Osmosis' ibc-hooks implementation label Jan 16, 2024
@disconsolate
Copy link

Hi,
When is this branch planned to be merged into main?

@Reecepbcups
Copy link
Member

@disconsolate Once it is finished & the Osmosis team reviews

@alexanderbez alexanderbez changed the title core(deps): Upgrade ibc-hooks to IBC v8 and SDK v0.50.x #152 core(deps): Upgrade ibc-hooks to IBC v8 and SDK v0.50.x Jan 29, 2024
@disconsolate
Copy link

Still waiting for this PR to be merged. Is there any estimation on when it will be done?

@faddat
Copy link
Contributor

faddat commented Mar 6, 2024

@disconsolate not really. Both @alexanderbez and I took a swing at it.

giving it another go here:

I think the trick will be making sure tests are done right.

@Reecepbcups
Copy link
Member

Reecepbcups commented May 22, 2024

Closing this in favor of PR #166 which builds off it and passed CI test. Looks to be mostly just bumps and tweaks.

Have tagged osmosis as reviewers + added a new review me label as well

Thanks bez!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ibc-hooks Label for items related to Osmosis' ibc-hooks implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants