Luis feedback

[joelamouche]

You will have to account for the fact the the User should input the ETH amount they want to deposit into Maker.
You should also verify if the user already has a DSA deployed and then not deploy one if that is the case.
You should also verify if the User has already set GelatoCore as auth on DSA and only cast a spell to do so, if the user hasnt done so already.
As a final part of the MakerDAI->CDAI spell I would also add another Action. Namely to Transfer the CDAI out from the DSA into the User wallet.
So that we don't have any CDAI stuck in our DSA.
Also you need to have buttons to withdraw any remaining ETH from Gelato via your DSA and back to the User's wallet.
Also any remaining ETH in your DSA should have a withdraw button.
You can combine both transfers into one by using DSA spells
check out this code for how to withdraw stuff from the DSA: https://github.com/gelatodigital/gelato-instadapp/blob/7295ad39fe155c1ae00bdd752d443fa167fa2d37/test/0_setup-DSA-Gelato.test.js#L140