refactor(tangle-dapp): Integrate bridge ERC-20 tokens into restaking #2783
Conversation
✅ Deploy Preview for tangle-dapp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for tangle-cloud ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for tangle-dapp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for tangle-cloud ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Progress:
|
yurixander
requested review from
AtelyPham,
devpavan04 and
monaiuu
as code owners
|
@yurixander I was able to bridge
CleanShot.2025-02-01.at.12.23.24.mp4As we discussed, these issues need to be addressed:
Otherwise, LGTM! |
| @@ -45,7 +45,7 @@ export default function usePayouts(): UsePayoutsReturnType { | |||
| LocalStorageKey.PAYOUTS, | |||
| ); | |||
|
|
|||
| const { rpcEndpoint, network } = useNetworkStore(); | |||
There was a problem hiding this comment.
Why don't we use the selector here?
const wsRpcEndpoint = useNetworkStore(store => store.network.wsRpcEndpoint);| }), | ||
| ); | ||
| return Object.entries(delegatorInfo.deposits).map(([assetIdString]) => { | ||
| const availableForWithdrawal = calculateRestakeAvailableBalance( |
There was a problem hiding this comment.
The backend has a field called delegated_amount that indicates the total delegated amount, which can be used to calculate the available balance.
This issue has been addressed in PR #2788.
| (delegation) => delegation.assetId === selectedAsset.id, | ||
| )?.amountBonded ?? ZERO_BIG_INT; | ||
|
|
||
| const availableBalance = amountBigInt - delegatedBigInt; |
There was a problem hiding this comment.
The backend has a field called delegated_amount that indicates the total delegated amount, which can be used to calculate the available balance.
This issue has been addressed in PR #2788.
| (delegation) => delegation.assetId === assetId, | ||
| )?.amountBonded ?? ZERO_BIG_INT; | ||
|
|
||
| const balance = new BN((amount - delegatedAmount).toString()); |
There was a problem hiding this comment.
The backend has a field called delegated_amount that indicates the total delegated amount, which can be used to calculate the available balance.
This issue has been addressed in PR #2788.
There was a problem hiding this comment.
@AtelyPham I tested with these calculations, and everything seems to be in order, including after you deposit, delegate, withdraw, etc. all the values updated accordingly. Just to make sure, that's the same case for when you used delegated_amount?
There was a problem hiding this comment.
^ Also these tests were done with local dev on latest Tangle node changes, since testnet has different / incorrect values which seem to have been fixed on newer node updates.
| return {}; | ||
| } | ||
|
|
||
| const depositedAsset = Object.entries(delegatorInfo.deposits).find( | ||
| ([assetId]) => assetId === selectedAssetId, | ||
| const availableBalance = calculateRestakeAvailableBalance( |
There was a problem hiding this comment.
The backend has a field called delegated_amount that indicates the total delegated amount, which can be used to calculate the available balance.
This issue has been addressed in PR #2788.
|
Conflicts here |
Summary of changes
Provide a detailed description of proposed changes.
Proposed area of change
Put an
xin the boxes that apply.apps/tangle-dappapps/tangle-cloudlibs/tangle-shared-uilibs/webb-ui-componentsAssociated issue(s)
Specify any issues that can be closed from these changes (e.g.
Closes #233).Screen Recording
If possible provide screenshots and/or a screen recording of proposed change.