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

feat(vaults): add deploy and register for vaults #8

Merged
merged 3 commits into from
Jan 17, 2025

Conversation

mart1n-xyz
Copy link
Contributor

feat(vaults): add deploy and register for vaults

  • deploy vaults
  • register vault
  • hooks and checks for success
  • update list of vaults
  • update vault balances and total staked
  • UI update
  • more efficient timing of chain queries

@mart1n-xyz mart1n-xyz requested a review from 0x-r4bbit January 15, 2025 12:16
@mart1n-xyz mart1n-xyz linked an issue Jan 15, 2025 that may be closed by this pull request
@0x-r4bbit
Copy link
Collaborator

It looks like there's a few commit in there that already landed with #7 no?

@0x-r4bbit 0x-r4bbit force-pushed the 3-allow-users-to-create-staking-vaults branch from 0421608 to d2ca3fa Compare January 17, 2025 12:23
@0x-r4bbit
Copy link
Collaborator

@mart1n-xyz I've rebased your branch on top of latest main and also tested this locally.
Great work
Gonna squash into semantic commits

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@0x-r4bbit 0x-r4bbit force-pushed the 3-allow-users-to-create-staking-vaults branch from d2ca3fa to c250baf Compare January 17, 2025 12:28
@0x-r4bbit 0x-r4bbit merged commit cf1b351 into main Jan 17, 2025
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.

Allow users to create staking vaults
2 participants