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(RewardStreamerMP): rename user vars to account #62

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

0x-r4bbit
Copy link
Collaborator

Closes #59

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Added natspec comments?
  • Ran pnpm adorno?
  • Ran pnpm verify?

@0x-r4bbit 0x-r4bbit force-pushed the refactor/user-account branch from 45b2ec5 to 7c18c8b Compare October 18, 2024 12:43
@0x-r4bbit 0x-r4bbit merged commit ba26e7c into main Oct 21, 2024
7 checks passed
0x-r4bbit added a commit that referenced this pull request Oct 25, 2024
This commit introduces changes related to vault registrations in the
stake manager.

The stake manager needs to keep track of the vaults a users creates so
it can aggregate accumulated MP across vaults for any given user.

The `StakeVault` now comes with a `register()` function which needs to
be called to register itself with the stake manager. `StakeManager` has
a new `onlyRegisteredVault` modifier that ensures only registered vaults
can actually `stake` and `unstake`.

Closes #62
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Change UserInfo to AccountInfo
2 participants