yieldprotocol / app-v2

Yield Protocol V2 UI
1 stars 0 forks source link

remove frax logic for ticket 0901 #1233

Closed jacobryas4 closed 12 months ago

jacobryas4 commented 1 year ago
netlify[bot] commented 1 year ago

Deploy Preview for yield-v2-app ready!

Name Link
Latest commit ab8712c40b51213c2179d981fe89b6ba7c4b5735
Latest deploy log https://app.netlify.com/sites/yield-v2-app/deploys/651fab7edf4e4e000765a89d
Deploy Preview https://deploy-preview-1233--yield-v2-app.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

marcomariscal commented 1 year ago

@jacobryas4, do you have a tenderly link to the successful fork transaction.

Also, we probably don't want to have a hook for each asset's removal, but due to the idiosyncrasy of this particular action/bug, we can have this useRemoveLiquidity specific to frax, and remove the hook after the user removes their liquidity.

jacobryas4 commented 1 year ago

@jacobryas4, do you have a tenderly link to the successful fork transaction.

Also, we probably don't want to have a hook for each asset's removal, but due to the idiosyncrasy of this particular action/bug, we can have this useRemoveLiquidity specific to frax, and remove the hook after the user removes their liquidity.

here is the tenderly link you requested: https://dashboard.tenderly.co/Yield/v2/fork/27baa1e9-5af6-436c-a22e-996860efa4e1