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

Review and Deploy sdBAL vester #1533

Open
Tritium-VLK opened this issue Nov 29, 2024 · 5 comments
Open

Review and Deploy sdBAL vester #1533

Tritium-VLK opened this issue Nov 29, 2024 · 5 comments
Assignees

Comments

@Tritium-VLK
Copy link
Member

https://github.com/stake-dao/sd-token-vestor/tree/main

@Tritium-VLK Tritium-VLK self-assigned this Nov 29, 2024
@Tritium-VLK
Copy link
Member Author

Note from Warren at sdbal:
Hey @tritium_vlk

We did a final review, made a couple changes to optimize the rewards flow better in case we makes some changes in our side in the reward distributions

https://github.com/stake-dao/sd-token-vestor/tree/main

Added a small part in the readme on how to claim voting rewards

He 2x confirmed they call this done/it will work. Have done a first pass reiview myself and everything looks fine. Will take another look later. @gosuto-inzasheru can you also have a look, and then IMO it's probs fine to deploy the factory/start using it. I can check with StakeDAO about who deploys the factory.

@gosuto-inzasheru
Copy link
Collaborator

@Xeonus
Copy link
Collaborator

Xeonus commented Dec 16, 2024

Verify that delegation assumed upon deposit or does it have to be done explicitly?

@gosuto-inzasheru
Copy link
Collaborator

delegation is done on construction:

/// @notice Contract initializer
/// @param _beneficiary Address of the beneficiary that will be able to claim tokens
function initialize(address _beneficiary) public initializer {
    /// Initialize factory address.
    FACTORY = msg.sender;

    /// Initialize beneficiary.
    beneficiary = _beneficiary;

    /// Delegate to Stake DAO delegation.
    IDelegate(DELEGATION_REGISTRY).setDelegate(DELEGATION_SPACE, DELEGATION);
}

however, it can be updated by the manager if necessary:

/// @notice Update delegation to Stake DAO delegation
/// @param _delegate Address of the delegate
function updateDelegation(address _delegate) external onlyRole(MANAGER_ROLE) {
    IDelegate(DELEGATION_REGISTRY).setDelegate(DELEGATION_SPACE, _delegate);
}

@gosuto-inzasheru
Copy link
Collaborator

vests 1735417835; 2024-12-28T21:30:35 CET

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

No branches or pull requests

3 participants