yieldprotocol / app-v2

Yield Protocol V2 UI
1 stars 0 forks source link

fix: explicitly use series to derive action allowed #1218

Open marcomariscal opened 1 year ago

marcomariscal commented 1 year ago

trying to prevent bugs where the series is null for some reason, inaccurately preventing the action.

there's still some implicit series handling because we get selectedSeries some times from userState, which we should try to mitigate going forward. ie: if handling a vault, we should be getting the selected vault's associated series (more explicit/accurate), not just getting the selectedSeries from global state.

netlify[bot] commented 1 year ago

Deploy Preview for yield-v2-app ready!

Name Link
Latest commit b06d1ab92681b0c187eae1103ed9fcdc1769f91a
Latest deploy log https://app.netlify.com/sites/yield-v2-app/deploys/649484f215066c000866bfdd
Deploy Preview https://deploy-preview-1218--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 settings.