yearn/app-plugin-yearn

Donjon code audit

Opened this issue · 1 comments

adrienlacombe commented

Why reserve 0 screens if after the amount is displayed here.

Majorfi commented

The ZapETH contract only has a deposit() method without arguments. From what I saw, the Ledger system already "append" the value in ETH and the gas.

Because of this, we don't need an extra screen.

Maybe I missed something on the screen and would be happy to change it, but not sure what!

(cf: snapshot images for the test to see the flow)