Add Portfolio page #513
Add Portfolio page #513
Conversation
|
@jophish is attempting to deploy a commit to the Ubeswap Team on Vercel. A member of the Team first needs to authorize it. |
| return token && totalSupply ? new TokenAmount(token, totalSupply.toString()) : undefined | ||
| } | ||
|
|
||
| export function useTotalSupplies(tokens?: (Token | undefined)[]): { [tokenAddress: string]: TokenAmount | undefined } { |
There was a problem hiding this comment.
This useTotalSupplies hook is required in the useCalculateLPPortfolio hook, in order to eventually calculate the total value of a user's LP holdings; we can't use useTotalSupply there, since a user's LP holdings may change, and iterating over all LP tokens and calling useTotalSupply for each may violate React's invariant of hooks being called in the same order every render.
There was a problem hiding this comment.
(I also figure this hook could probably be useful elsewhere in the codebase!)
|
@BriungRi Bumping this here since this has been stale for a while; I believe you've been CC'd on an email thread relating to this work. This work is the result of an Ubeswap community grant; I would like to get this merged in and close out the grant, if still possible. |
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/ubeswap/ubeswap-interface/9cNCKgtFMitT7UZnXBmNT5MFHSDu |
|
Seeing bugs. Can you investigate?
|
|
@BriungRi Thanks; pushed some changes and should be good now. Let me know if there are other issues. |
|
@BriungRi Bumping on this again. |
2 participants
Adds in a Portfolio page for users to track their total holdings, LP/token spit, token distribution, and LP distribution.