Skip to content

feat: liquid staking module integration#61

Merged
tgntr merged 125 commits into
TacBuild:mainfrom
pixelplex:migration
Oct 10, 2025
Merged

feat: liquid staking module integration#61
tgntr merged 125 commits into
TacBuild:mainfrom
pixelplex:migration

Conversation

@pixelplex
Copy link
Copy Markdown
Contributor

Integration of "x/liquidstake" module

Comment thread app/upgrades.go Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i see you created a new init file, can we adjust init.sh instead and avoid having multiple init files?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We made it to initialize multiply liquid validators in comparison to standard init. But i'm sure we could generalize original init script enough, if it's necessary. We will make this when upcoming dependency changes will be ready.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed.

Comment thread go.mod
Comment thread app/upgrades/v1.0.2/upgrades.go
Comment thread go.mod Outdated
@tgntr
Copy link
Copy Markdown
Collaborator

tgntr commented Oct 10, 2025

could you please rename your PR to something more descriptive like feat: liquid staking

Comment thread utils/eth.go Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm thinking about moving whole function to upgrade handler.

@pixelplex pixelplex changed the title Migration feat: liquid staking module integration Oct 10, 2025
Comment thread go.mod
@tgntr tgntr merged commit 4c81ca8 into TacBuild:main Oct 10, 2025
6 checks passed
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.

6 participants