[WIP] Feature/earn new #900
Conversation
…rmation and upgrade options
…th styled elements
…kground transition
…get into feature/earn-new
Preview stand statusStand was demolished |
Signed-off-by: Alexander Khramov <[email protected]>
Signed-off-by: Alexander Khramov <[email protected]>
Signed-off-by: Alexander Khramov <[email protected]>
Signed-off-by: Alexander Khramov <[email protected]>
Signed-off-by: Alexander Khramov <[email protected]>
|
|
||
| const isEarnListPage = router.pathname === EARN_PATH; | ||
| const isEarnVault = router.pathname.includes(`${EARN_PATH}/[vault]/[action]`); | ||
| const isEarnVaultV1 = |
There was a problem hiding this comment.
TODO: move this logic to layout props
| <TableGroup> | ||
| <TableItem> | ||
| <TableLabel>Curator</TableLabel> | ||
| <TableValue>● Mellow</TableValue> |
| @@ -0,0 +1,252 @@ | |||
| export const ETH_DEPOSIT_QUEUE_DVSTETH_ABI = [ | |||
There was a problem hiding this comment.
Deposit queues have same ABIs?
There was a problem hiding this comment.
or at least part that's used by us is the same
| @@ -0,0 +1,260 @@ | |||
| export const ETH_REDEEM_QUEUE_WSTETH_ABI = [ | |||
There was a problem hiding this comment.
also feels like they are the same
| wETH: { | ||
| balance: wethBalanceQuery.data, | ||
| }, | ||
| GG: { |
There was a problem hiding this comment.
we don't need this in form I think, updgrade will be done separtely
| 'collect', | ||
| collector.address, | ||
| vault.address, | ||
| { address: address ?? null }, |
| address, // account | ||
| vault.address, // vault | ||
| COLLECTOR_CONFIG, // config | ||
| ])) as CollectResponse; |
There was a problem hiding this comment.
what's wrong with viem types?
| import { getSTGCollectorContract, getSTGVaultContract } from '../contracts'; | ||
| import { STG_COLLECTOR_CONFIG } from '../consts'; | ||
|
|
||
| export type STGCollectResponse = { |
There was a problem hiding this comment.
TODO: use common collector hook
| @@ -0,0 +1,16 @@ | |||
| export const ETH_TOKEN_SYMBOL = 'earnETH'; | |||
There was a problem hiding this comment.
Let's switch naming ETH_VAULT, ETH_ is too ambigous for stake widget
| const depositQueue = depositQueueGetter({ | ||
| publicClient: core.rpcProvider, | ||
| walletClient: core.web3Provider as WalletClient, | ||
| token, |
There was a problem hiding this comment.
for us to get correct depositQueue seems token is not eneough, we should also specify vault, right?
There was a problem hiding this comment.
I think we can add vault: 'str' | 'eth" ... param to have general getter
There was a problem hiding this comment.
token is enough because we get correct vault by token
3 participants
Description
WIP new earn
Demo
Code review notes
Testing notes
Checklist: