yearn / yearn.fi

https://yearn.fi
GNU General Public License v3.0
22 stars 48 forks source link

Chore: Ensure safeChainID #124

Closed Majorfi closed 1 year ago

Majorfi commented 1 year ago

What happened 🆘

The behavior between safeChainID and vaultChainID can be unexpected when the user is connected on mainnet (for ex) but on the page for a vault on optimism (for ex). For actions related to this vault chainID, we need to ensure we are using the vaultChainID rather than the safeChainID

To Reproduce 🔂

NA

More 🔎

NA

Majorfi commented 1 year ago

See comments in #123

karelianpie commented 1 year ago

TODO: Disabling the action buttons when the user is in a different network than the vault

Done with https://github.com/yearn/yearn.fi/pull/138