rtoken-project / rtoken-monorepo

RToken Ethereum monorepo
https://rdai.money
89 stars 43 forks source link

Implement mintFor using DAI's permit function #36

Closed hellwolf closed 3 years ago

hellwolf commented 4 years ago

https://medium.com/@Degens/betting-without-eth-dais-new-permit-feature-5517293f3246

"This new feature allows the approval process to be done by a separate party, same as our bet on the Rams above" / A meta transaction for approve if you will.

ACs:

Bounty size: 100 rĐ

hellwolf commented 4 years ago

@pi0neerpat do you mind if we take this as a bounty for 100 rĐ ?

pi0neerpat commented 4 years ago

Sure go ahead. After research from rTrees, I concluded that this was actually not very valuable. This would require a relayer (could use mosendo) for the approval, but the user still needs gas ETH for any rtoken function call like mint.

If you wanted to get real fancy, you take the user's dai, mint rDAI in another account, and send it to the user. But that's getting into more "product" territory and away from core rToken functionality

hellwolf commented 4 years ago

Sure go ahead. After research from rTrees, I concluded that this was actually not very valuable. This would require a relayer (could use mosendo) for the approval, but the user still needs gas ETH for any rtoken function call like mint.

If you wanted to get real fancy, you take the user's dai, mint rDAI in another account, and send it to the user. But that's getting into more "product" territory and away from core rToken functionality

Yes, you are right, the mintFor, mintForWithSelectedhat and mintForWithNewHat functions.

pi0neerpat commented 4 years ago

That would work

pi0neerpat commented 3 years ago

closed by #45