Skip to content

Deploy and configure trustee fee collection#543

Merged
franckc merged 5 commits into
masterfrom
franck/trustee
Feb 3, 2021
Merged

Deploy and configure trustee fee collection#543
franckc merged 5 commits into
masterfrom
franck/trustee

Conversation

@franckc
Copy link
Copy Markdown

@franckc franckc commented Feb 2, 2021

Deploy script 13 to deploy VaultCore that implements the trustee fees and to configure the trustee fee and address

  • address is for now the multi-sig strategist at 0xF14BBdf064E3F67f51cd9BD646aE3716aD938FDC
  • fee is set to 10%

Also added a helper hardhat task for artificially generating yield on the vault - handy for testing.

I tested on the fork that the trustee account gets yield upon a rebase, using the following sequence:

FORK=true npx hardhat yield --network localhost
FORK=true npx hardhat balance --account 0xF14BBdf064E3F67f51cd9BD646aE3716aD938FDC --network localhost
FORK=true npx hardhat rebase --network localhost
FORK=true npx hardhat balance --account 0xF14BBdf064E3F67f51cd9BD646aE3716aD938FDC --network localhost

@franckc franckc changed the title configuration for the trustee Deploy and configure trustee fee collection Feb 3, 2021
@franckc
Copy link
Copy Markdown
Author

franckc commented Feb 3, 2021

We missed including VaultCore upgrade in deploy-012 so I will add it to this PR... @tomlinton please hold on reviewing this PR. it is not ready.

@franckc
Copy link
Copy Markdown
Author

franckc commented Feb 3, 2021

@tomlinton @DanielVF This is now ready for review... Thanks :)

@franckc franckc merged commit d2231cd into master Feb 3, 2021
@franckc franckc deleted the franck/trustee branch February 3, 2021 19:08
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.

2 participants