-
Notifications
You must be signed in to change notification settings - Fork 195
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
[SI][Vaults] feat: Dashboard UX updates #915
Changes from 34 commits
b6156ad
a11d6b6
29ef4ad
b5c839f
4b16505
41f6125
ae59476
de3d3b9
3261a8c
c228afd
c6cc70f
739a60d
839b726
99f7d9a
186e266
0aea721
c4e7ceb
62710b0
95b11fb
b5ce3a4
fe87ca3
50259ec
fac1f9d
710ccac
77d8739
0903095
5300444
f24c48e
90f64d4
5a907fc
b529f7b
5215e35
f78235c
bea9a49
616a0f8
2b871fb
c9c7f74
e470a89
4ff312b
34d4519
b7e5536
0c2c170
067f38b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
folkyatina marked this conversation as resolved.
Show resolved
Hide resolved
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please update following comment:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. fixed There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. votingCommittee -> multiConfirmationRoles There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. reserveRatio() -> reserveRatioBP() or at least comment that this is the BP convention There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. fixed There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. let's document whether already minted are included or not
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. fixed |
Large diffs are not rendered by default.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. keccak256("Vault.Delegation.CuratorRole") ⇒ keccak256("vaults.Delegation.CuratorRole") and so on There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. let's remove There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. vote -> multiConfirmationRole (everywhere) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. let's move this one to 0.8.25 and take recovery lib from CSM |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
// SPDX-License-Identifier: UNLICENSED | ||
// for testing purposes only | ||
|
||
pragma solidity 0.8.25; | ||
|
||
import {ERC721} from "@openzeppelin/contracts-v5.2/token/ERC721/ERC721.sol"; | ||
|
||
contract ERC721_MockForDashboard is ERC721 { | ||
constructor() ERC721("MockERC721", "M721") {} | ||
|
||
function mint(address _recipient, uint256 _tokenId) external { | ||
_mint(_recipient, _tokenId); | ||
} | ||
} |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -41,19 +41,27 @@ contract VaultHub__MockForDashboard { | |||||
emit Mock__VaultDisconnected(vault); | ||||||
} | ||||||
|
||||||
function mintSharesBackedByVault(address /* vault */, address recipient, uint256 amount) external { | ||||||
function mintSharesBackedByVault(address vault, address recipient, uint256 amount) external { | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
steth.mintExternalShares(recipient, amount); | ||||||
vaultSockets[vault].sharesMinted = uint96(vaultSockets[vault].sharesMinted + amount); | ||||||
} | ||||||
|
||||||
function burnSharesBackedByVault(address /* vault */, uint256 amount) external { | ||||||
steth.burnExternalShares(amount); | ||||||
function burnSharesBackedByVault(address _vault, uint256 _amountOfShares) external { | ||||||
if (_vault == address(0)) revert ZeroArgument("_vault"); | ||||||
if (_amountOfShares == 0) revert ZeroArgument("_amountOfShares"); | ||||||
steth.burnExternalShares(_amountOfShares); | ||||||
vaultSockets[_vault].sharesMinted = uint96(vaultSockets[_vault].sharesMinted - _amountOfShares); | ||||||
} | ||||||
|
||||||
function voluntaryDisconnect(address _vault) external { | ||||||
emit Mock__VaultDisconnected(_vault); | ||||||
} | ||||||
|
||||||
function rebalance() external payable { | ||||||
vaultSockets[msg.sender].sharesMinted = 0; | ||||||
|
||||||
emit Mock__Rebalanced(msg.value); | ||||||
} | ||||||
|
||||||
error ZeroArgument(string argument); | ||||||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thresholdReserveRatio vs reserveRatioThreshold is a bit weird